Re: Displaced c++-style comments in net-snmp-5.3.0.1-sourcecode
Dave Shield <[email protected]>
| Newsgroups | gmane.network.net-snmp.user |
|---|---|
| Message-ID | <[email protected]> |
On Tue, 2006-03-07 at 16:41 -0800, Wes Hardaker wrote: > >>>>> On Wed, 22 Feb 2006 11:15:05 +0000, Dave Shield <[email protected]> said: > > >> I noticed some c++-style comments ("// code") in some > >> files in ./agent/mibgroup/disman/event/mte*. > > Dave> Yes - those look like temporary removals of code that > Dave> should have worked but had been causing problems. > Dave> I tend to use C++-style comments for those, to distinguish > Dave> them from "real" comments, and remind me that the problem > Dave> still needs to be addressed. > > To prevent this we shouldn't ever check in code with // comments in > it. Period. Agreed. These are meant to be temporary comments in my local working code. > If you need a reminder, use an XXX ... Oh, I do - frequently. Part of the reason for using the // style is to remind me that a particular change should *not* be checked in until the issue has been sorted (or at least thought about a bit more). Unfortunately that process failed this time. Mea culpa. Dave ------------------------------------------------------- This SF.Net email is sponsored by xPML, a groundbreaking scripting language that extends applications into web and mobile media. Attend the live webcast and join the prime developer group breaking into this new coding territory! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642 _______________________________________________ Net-snmp-users mailing list [email protected] Please see the following page to unsubscribe or change other options: https://lists.sourceforge.net/lists/listinfo/net-snmp-users