{"comment":{"url":"https://api.github.com/repos/devinacker/bsnes-plus/comments/10787626","html_url":"https://github.com/devinacker/bsnes-plus/commit/02c626a77e119ce4be18e1b954f593a873d2c5ee#commitcomment-10787626","id":10787626,"user":{"login":"awjackson","id":9252545,"avatar_url":"https://avatars.githubusercontent.com/u/9252545?v=3","gravatar_id":"","url":"https://api.github.com/users/awjackson","html_url":"https://github.com/awjackson","followers_url":"https://api.github.com/users/awjackson/followers","following_url":"https://api.github.com/users/awjackson/following{/other_user}","gists_url":"https://api.github.com/users/awjackson/gists{/gist_id}","starred_url":"https://api.github.com/users/awjackson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/awjackson/subscriptions","organizations_url":"https://api.github.com/users/awjackson/orgs","repos_url":"https://api.github.com/users/awjackson/repos","events_url":"https://api.github.com/users/awjackson/events{/privacy}","received_events_url":"https://api.github.com/users/awjackson/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"02c626a77e119ce4be18e1b954f593a873d2c5ee","created_at":"2015-04-18T09:31:50Z","updated_at":"2015-04-18T09:31:50Z","body":"The change to the offset calculation was simply reversing the sign, because storing negative numbers in an unsigned array and relying on arithmetic wraparound (in Bus::map() `addr` is always greater or equal than `offset`) is weird and a bit unsafe."}}