Hacker News new | ask | show | jobs
by stanferder 2323 days ago
> The prefix/postfix change is a red herring, that doesn't do anything

The prefix/postfix change does something to the reader. It's a distraction trying to figure out if that change was intended, whether the author knows it has no effect, or whether I'm wrong about the change having no effect.

I appreciate the author doing the hard work of fixing this, especially with the code in full view of the public, but if I were an official reviewer I'd ask to get all the unnecessary changes removed.