3 ms·
I never understood this line of reasoning. If you're changing a line of code you should at least look at the whole function that contains it, and update the com
by tomerv 5y ago
I never understood this line of reasoning. If you're changing a line of code you should at least look at the whole function that contains it, and update the comments according to the change.
> given they don't have to change them to meet the requirements
What are the requirements? Code review is standard practice, so I see clear code as part of the requirements.
- morbia 5y agoDevelopers and code reviewers make mistakes. If a change involves looking at code numerous trivial comments, the probability that both the developer and the reviewers misses at least one increases. This is particularly problematic for this sort of issue because it's impossible for a static analysis tool to pick up.