{"comment":{"url":"https://api.github.com/repos/AnishShah/numpy/comments/15825058","html_url":"https://github.com/AnishShah/numpy/commit/bfdd6c7c0be11bbe89f110d30034767550ce471f#commitcomment-15825058","id":15825058,"user":{"login":"madphysicist","id":4617010,"avatar_url":"https://avatars.githubusercontent.com/u/4617010?v=3","gravatar_id":"","url":"https://api.github.com/users/madphysicist","html_url":"https://github.com/madphysicist","followers_url":"https://api.github.com/users/madphysicist/followers","following_url":"https://api.github.com/users/madphysicist/following{/other_user}","gists_url":"https://api.github.com/users/madphysicist/gists{/gist_id}","starred_url":"https://api.github.com/users/madphysicist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/madphysicist/subscriptions","organizations_url":"https://api.github.com/users/madphysicist/orgs","repos_url":"https://api.github.com/users/madphysicist/repos","events_url":"https://api.github.com/users/madphysicist/events{/privacy}","received_events_url":"https://api.github.com/users/madphysicist/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"bfdd6c7c0be11bbe89f110d30034767550ce471f","created_at":"2016-02-02T11:36:15Z","updated_at":"2016-02-02T11:36:15Z","body":"  - On line 732/735, the check for `if cond:` is superfluous because it is already in a similar `if` statement.\r\n  - You should add a check that triggers the `ValueError` to make sure that works as expected.\r\n\r\nAside from that, LGTM. I am on a very crappy phone ATM, in case you are wondering why I can not make comments directly in the code."}}