{"comment":{"url":"https://api.github.com/repos/couchbaselabs/phosphor/comments/17611972","html_url":"https://github.com/couchbaselabs/phosphor/commit/b41d00cdf204d5d60a000b22d190b9f50deb0097#commitcomment-17611972","id":17611972,"user":{"login":"daverigby","id":755459,"avatar_url":"https://avatars.githubusercontent.com/u/755459?v=3","gravatar_id":"","url":"https://api.github.com/users/daverigby","html_url":"https://github.com/daverigby","followers_url":"https://api.github.com/users/daverigby/followers","following_url":"https://api.github.com/users/daverigby/following{/other_user}","gists_url":"https://api.github.com/users/daverigby/gists{/gist_id}","starred_url":"https://api.github.com/users/daverigby/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daverigby/subscriptions","organizations_url":"https://api.github.com/users/daverigby/orgs","repos_url":"https://api.github.com/users/daverigby/repos","events_url":"https://api.github.com/users/daverigby/events{/privacy}","received_events_url":"https://api.github.com/users/daverigby/received_events","type":"User","site_admin":false},"position":26,"line":42,"path":"src/trace_log.cc","commit_id":"b41d00cdf204d5d60a000b22d190b9f50deb0097","created_at":"2016-05-25T12:32:03Z","updated_at":"2016-05-25T12:32:03Z","body":"re: default case, if you remove this case from the switch, make the expected cases return in their case, and instead just throw std::invalid_argument outside then the compiler can warn if a new enumeration value is later added (and isn't handled here)."}}
{"actor":{"display_login":"daverigby"}}