>From: [EMAIL PROTECTED] (Raphael Manfredi)
>
>This is very neat, and it has been in my TODO for a long time...
>
>One comment on your patch though: it lacks comments. I understand the
>fix is simple, but when you read the code, nowhere does it say what it
>does.
>
>I'd summarize what you said in your message as comments in the code, so
>that
>someone who takes a fresh view of the code can actually read the comments
>and mentally reverse-engineer the code only if necessary (bug fixing or
>refactoring).
>
>Raphael
I'll have to remember that. I keep bouncing from one extreme to the other
in that regard - professionally, I tend to add TONS of comments, as most
shops I've worked at required such, while in the OSS arena I keep getting
messages to "not clutter up the code with unneeded comments". I guess it's
a matter of the project style....
Lloyd Bryant
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Gtk-gnutella-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/gtk-gnutella-devel
