{"comment":{"url":"https://api.github.com/repos/yast/yast-yast2/comments/9299729","html_url":"https://github.com/yast/yast-yast2/commit/061aa1edade2c1c6f7d02b38e1ce7d212309925d#commitcomment-9299729","id":9299729,"user":{"login":"mvidner","id":102056,"avatar_url":"https://avatars.githubusercontent.com/u/102056?v=3","gravatar_id":"","url":"https://api.github.com/users/mvidner","html_url":"https://github.com/mvidner","followers_url":"https://api.github.com/users/mvidner/followers","following_url":"https://api.github.com/users/mvidner/following{/other_user}","gists_url":"https://api.github.com/users/mvidner/gists{/gist_id}","starred_url":"https://api.github.com/users/mvidner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mvidner/subscriptions","organizations_url":"https://api.github.com/users/mvidner/orgs","repos_url":"https://api.github.com/users/mvidner/repos","events_url":"https://api.github.com/users/mvidner/events{/privacy}","received_events_url":"https://api.github.com/users/mvidner/received_events","type":"User","site_admin":false},"position":4,"line":389,"path":"library/network/src/modules/CWMFirewallInterfaces.rb","commit_id":"061aa1edade2c1c6f7d02b38e1ce7d212309925d","created_at":"2015-01-15T16:11:13Z","updated_at":"2015-01-15T16:11:13Z","body":"But this may omit some side effects that other code depends on. I would keep this in and just drop the return value."}}