{"comment":{"url":"https://api.github.com/repos/ripple/rippled/comments/11183060","html_url":"https://github.com/ripple/rippled/commit/a561c72b7577311f3ad323ffb69baab9e577155a#commitcomment-11183060","id":11183060,"user":{"login":"ximinez","id":6757714,"avatar_url":"https://avatars.githubusercontent.com/u/6757714?v=3","gravatar_id":"","url":"https://api.github.com/users/ximinez","html_url":"https://github.com/ximinez","followers_url":"https://api.github.com/users/ximinez/followers","following_url":"https://api.github.com/users/ximinez/following{/other_user}","gists_url":"https://api.github.com/users/ximinez/gists{/gist_id}","starred_url":"https://api.github.com/users/ximinez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ximinez/subscriptions","organizations_url":"https://api.github.com/users/ximinez/orgs","repos_url":"https://api.github.com/users/ximinez/repos","events_url":"https://api.github.com/users/ximinez/events{/privacy}","received_events_url":"https://api.github.com/users/ximinez/received_events","type":"User","site_admin":false},"position":106,"line":376,"path":"src/ripple/shamap/impl/SHAMapSync.cpp","commit_id":"a561c72b7577311f3ad323ffb69baab9e577155a","created_at":"2015-05-13T19:14:10Z","updated_at":"2015-05-13T19:14:10Z","body":"Depth is unsigned, so it can't be <0. If it's 0, but bc is 1, that's exactly the intention of this check. I'm 99.9% sure that we can't have underflows, either because of the combined checks."}}