{"comment":{"url":"https://api.github.com/repos/mdmillerii/openbmc/comments/16464076","html_url":"https://github.com/mdmillerii/openbmc/commit/d7b6a57f89618f18447f9134f51eebe7a3b516cd#commitcomment-16464076","id":16464076,"user":{"login":"amboar","id":526481,"avatar_url":"https://avatars.githubusercontent.com/u/526481?v=3","gravatar_id":"","url":"https://api.github.com/users/amboar","html_url":"https://github.com/amboar","followers_url":"https://api.github.com/users/amboar/followers","following_url":"https://api.github.com/users/amboar/following{/other_user}","gists_url":"https://api.github.com/users/amboar/gists{/gist_id}","starred_url":"https://api.github.com/users/amboar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/amboar/subscriptions","organizations_url":"https://api.github.com/users/amboar/orgs","repos_url":"https://api.github.com/users/amboar/repos","events_url":"https://api.github.com/users/amboar/events{/privacy}","received_events_url":"https://api.github.com/users/amboar/received_events","type":"User","site_admin":false},"position":11,"line":69,"path":"meta-phosphor/common/recipes-phosphor/obmc-phosphor-initfs/files/obmc-update.sh","commit_id":"d7b6a57f89618f18447f9134f51eebe7a3b516cd","created_at":"2016-03-03T03:13:17Z","updated_at":"2016-03-03T03:13:17Z","body":"Should we move the shift calls out of the case statement to the bottom of the loop body? That should remove some noise (mainly in future patches that introduce more options)"}}