{"comment":{"url":"https://api.github.com/repos/deads2k/origin/comments/9791272","html_url":"https://github.com/deads2k/origin/commit/c82b7b334930a65a9e2b4c4e2212a590511348b4#commitcomment-9791272","id":9791272,"user":{"login":"smarterclayton","id":1163175,"avatar_url":"https://avatars.githubusercontent.com/u/1163175?v=3","gravatar_id":"","url":"https://api.github.com/users/smarterclayton","html_url":"https://github.com/smarterclayton","followers_url":"https://api.github.com/users/smarterclayton/followers","following_url":"https://api.github.com/users/smarterclayton/following{/other_user}","gists_url":"https://api.github.com/users/smarterclayton/gists{/gist_id}","starred_url":"https://api.github.com/users/smarterclayton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/smarterclayton/subscriptions","organizations_url":"https://api.github.com/users/smarterclayton/orgs","repos_url":"https://api.github.com/users/smarterclayton/repos","events_url":"https://api.github.com/users/smarterclayton/events{/privacy}","received_events_url":"https://api.github.com/users/smarterclayton/received_events","type":"User","site_admin":false},"position":47,"line":47,"path":"pkg/authorization/escalation/policy_comparator.go","commit_id":"c82b7b334930a65a9e2b4c4e2212a590511348b4","created_at":"2015-02-18T01:06:45Z","updated_at":"2015-02-18T01:06:45Z","body":"In general invoking List() should only be for type conversion into another form for return, because it's a memory allocation and you've already paid the cost to create the map, and you can range / find the items as easily."}}