{"comment":{"url":"https://api.github.com/repos/vodik/sipp/comments/11421759","html_url":"https://github.com/vodik/sipp/commit/2961baa26c25f91e9d2a599b3c4940b1aab1f7d4#commitcomment-11421759","id":11421759,"user":{"login":"wdoekes","id":1225014,"avatar_url":"https://avatars.githubusercontent.com/u/1225014?v=3","gravatar_id":"","url":"https://api.github.com/users/wdoekes","html_url":"https://github.com/wdoekes","followers_url":"https://api.github.com/users/wdoekes/followers","following_url":"https://api.github.com/users/wdoekes/following{/other_user}","gists_url":"https://api.github.com/users/wdoekes/gists{/gist_id}","starred_url":"https://api.github.com/users/wdoekes/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wdoekes/subscriptions","organizations_url":"https://api.github.com/users/wdoekes/orgs","repos_url":"https://api.github.com/users/wdoekes/repos","events_url":"https://api.github.com/users/wdoekes/events{/privacy}","received_events_url":"https://api.github.com/users/wdoekes/received_events","type":"User","site_admin":false},"position":5,"line":7,"path":"Makefile.am","commit_id":"2961baa26c25f91e9d2a599b3c4940b1aab1f7d4","created_at":"2015-05-29T07:13:42Z","updated_at":"2015-05-29T07:13:42Z","body":"Don't add this into the tree. I'm not sure what other compilers say. If you really want it included (I'd go for -Wall), then I'd like to see some autoconf to enable it if available.\r\n\r\nOther than that, this commit looks good to go.\r\n\r\n(Although I'd personally fix the spaces in every line I'd touch, so we won't see another commit touching those once more later on.)"}}