[merraksh]

I always thought this process was sort-of intended. I feel it shows some respect to the author of the code under review to comment on it, and not to change it–especially not if that would result in changes that are then merged without first getting approval from the author.

Agreed. It also prevents the reviewer from introducing extra bugs from knowing the code less than the author.