Ah, the agony. :grinning:

I would probably change it anyway. But then again, I've been known to do things like rename a local variable to `annote_id` to shave off three letters so a line will fit in 80 cols, which, when I say it out load, seems a little wacky.

You can also iterate until `git_note_next` returns any non-zero value instead of checking explicitly for `GIT_ITEROVER` (because if you get another error you should stop the loop, too).  You could store the return value in a local `error` variable and after the loop is over, you can assert that is == to GIT_ITEROVER.

Or you can leave it as is. I wouldn't not merge the PR over something like this.