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
