Re:Docker for Windows won't run if Razer Synapse driver mana...
by inetknght on Tuesday February 18, 2020 @09:02AM
(#22356096)
Especially if it's not a copy-paste job. If there's any adjustments or even a rewrite, the comment should include the reason why the changes were made. The next guy coming along might not realize that changes were needed
[ Parent
| on HN ]
Re:Docker for Windows won't run if Razer Synapse driver mana...
by nradov on Tuesday February 18, 2020 @12:58PM
(#22358306)
It's usually better to put that information in the revision comments when you commit rather than in the code itself. Otherwise the code accumulates layers and layers of comments over years.
Re:Docker for Windows won't run if Razer Synapse driver mana...
by jonreem on Tuesday February 18, 2020 @04:52PM
(#22360771)
Personally, I've developed a strong preference for putting this type of information as close to the affected code as possible. Immediately next to the affected code is the only place these types of comments have a prayer of being kept up to date. Keeping them in another document, a commit message, or anything similar just results in the content of the message instantly going out of date.
Re:Docker for Windows won't run if Razer Synapse driver mana...
by penagwin on Tuesday February 18, 2020 @04:47PM
(#22360738)
I feel like it depends - I know I don't just start digging through the commit history of every line I edit.
If I'm trying to fix a bug, and only have to change one line for it to start working, and the tests still pass - but that line was there for a reason that wasn't commented (some edge case) - I'll likely miss it.