:On Wed, 28 Jul 1999, Nate Williams wrote:
:> >
:> > These were changes that were necessary to make ipfw readable enough that
:> > I could work with it in this area. They aren't just to clean it up, or
:> > just for change's sake. They need to stay in.
:>
:> C'mon now, re-ording the lines is *certainly* not necessary to work.
:
:That's true. I sure didn't do that.
:
:>
:> *rant on*
:> Brian, FreeBSD isn't your private playground for playing around, this is
:> a group project, and you gotta follow the rules, or you don't get to
:> play with the rest of the folks....
:
:The rules don't say "leave the code that you work with in a bigger mess than
:when you started." Cleaning up code is a fact of life, and it _NEEDS_ to be
:done to get work done, very often. You have to learn to deal with that.
:
:> *rant off*
:>
:> Nate
:>
: Brian Fundakowski Feldman _ __ ___ ____ ___ ___ ___
: [EMAIL PROTECTED] _ __ ___ | _ ) __| \
Side note: I do the same thing: clean up certain portions of the code
in order to make them more readable while I'm working on the module. And
while I tend to agree that it would be nice to commit the bug fixes
separately, it just isn't practical most of the time because one might
have already spent too big a chunk of one's own time to do the work in
the first place.
And I get the same sort of crap from certain core members, too. It's
truely annoying when you've spent the time ( sometimes a whole LOT of
time ) and trouble to fix something and all some people can do is
complain about extremely trivial elements of the work.
-Matt
Matthew Dillon
<[EMAIL PROTECTED]>
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message