Re:Document the Why
by geoka9 on Wednesday March 06, 2013 @02:41PM
(#5333302)
going to get out of touch with the code, unless you have some phase where you are going to review them in depth
I think treating comments as something that's done in a separate phase is the wrong mindset. Comments should be integral part of code and therefore change whenever the relevant code changes.
Re:Document the Why
by mercurial on Wednesday March 06, 2013 @03:17PM
(#5333540)
Maybe, but it's not that easy. You're generally going to see a 3-lines-of-context diff, and the reviewer won't necessarily have the idea to read the entire file (or worse, go poking elsewhere in the codebase) to see if comments deserve changing.
I think treating comments as something that's done in a separate phase is the wrong mindset. Comments should be integral part of code and therefore change whenever the relevant code changes.