{"comment":{"url":"https://api.github.com/repos/HughMacdonald/gaffer/comments/13361059","html_url":"https://github.com/HughMacdonald/gaffer/commit/9a65a0f7e273c282fdb558c4ec38be8167293e74#commitcomment-13361059","id":13361059,"user":{"login":"johnhaddon","id":1133871,"avatar_url":"https://avatars.githubusercontent.com/u/1133871?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhaddon","html_url":"https://github.com/johnhaddon","followers_url":"https://api.github.com/users/johnhaddon/followers","following_url":"https://api.github.com/users/johnhaddon/following{/other_user}","gists_url":"https://api.github.com/users/johnhaddon/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhaddon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhaddon/subscriptions","organizations_url":"https://api.github.com/users/johnhaddon/orgs","repos_url":"https://api.github.com/users/johnhaddon/repos","events_url":"https://api.github.com/users/johnhaddon/events{/privacy}","received_events_url":"https://api.github.com/users/johnhaddon/received_events","type":"User","site_admin":false},"position":8,"line":107,"path":"include/GafferImage/ImageAlgo.inl","commit_id":"9a65a0f7e273c282fdb558c4ec38be8167293e74","created_at":"2015-09-22T08:21:57Z","updated_at":"2015-09-22T08:21:57Z","body":"Maybe just `channelData.begin() + sampleOffsets[pixelId]`? From my understanding, `boost::next()` is useful for types (non-random-access iterators) where that operation isn't available, so by using it here we're implying a limitation that doesn't exist for our type, and making the code harder to follow."}}