{"comment":{"url":"https://api.github.com/repos/TheUnderscores/protolang/comments/9165729","html_url":"https://github.com/TheUnderscores/protolang/commit/cefe42c2ab91bdc03176a42a7afc1119b579df33#commitcomment-9165729","id":9165729,"user":{"login":"JoshuaBrockschmidt","id":10371153,"avatar_url":"https://avatars.githubusercontent.com/u/10371153?v=3","gravatar_id":"","url":"https://api.github.com/users/JoshuaBrockschmidt","html_url":"https://github.com/JoshuaBrockschmidt","followers_url":"https://api.github.com/users/JoshuaBrockschmidt/followers","following_url":"https://api.github.com/users/JoshuaBrockschmidt/following{/other_user}","gists_url":"https://api.github.com/users/JoshuaBrockschmidt/gists{/gist_id}","starred_url":"https://api.github.com/users/JoshuaBrockschmidt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JoshuaBrockschmidt/subscriptions","organizations_url":"https://api.github.com/users/JoshuaBrockschmidt/orgs","repos_url":"https://api.github.com/users/JoshuaBrockschmidt/repos","events_url":"https://api.github.com/users/JoshuaBrockschmidt/events{/privacy}","received_events_url":"https://api.github.com/users/JoshuaBrockschmidt/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"cefe42c2ab91bdc03176a42a7afc1119b579df33","created_at":"2015-01-06T08:45:00Z","updated_at":"2015-01-06T08:45:00Z","body":"Trailing whitespace? What do you mean?\r\n\r\nI may just convert it to a table of functions. I don't like having holes in\r\nthe array. Kind of messy, I guess.\r\nOn Jan 5, 2015 10:57 PM, \"Robert Cochran\" <notifications@github.com> wrote:\r\n\r\n> Nothing looks obviously wrong here. You did manage to add 'trailing\r\n> whitespace' (src/main.c:31) though. Check to make you don't leave that in\r\n> your\r\n> source ('git diff' helpfully highlights trailing whitespace in red) Also,\r\n> if we know that strings will be measured via strlen(), then I think there\r\n> isn't a need to have a slot for it in typeSizes. (src/vars.c:14-15)\r\n> Otherwise this patch looks good to me.\r\n>\r\n> —\r\n> Reply to this email directly or view it on GitHub\r\n> <https://github.com/TheUnderscores/protolang/commit/cefe42c2ab91bdc03176a42a7afc1119b579df33#commitcomment-9165026>\r\n> .\r\n>"}}