Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 28 Jul 1999 16:32:58 -0400 (EDT)
From:      Alfred Perlstein <bright@rush.net>
To:        "Brian F. Feldman" <green@FreeBSD.ORG>
Cc:        Nate Williams <nate@mt.sri.com>, hackers@FreeBSD.ORG, freebsd-ipfw@FreeBSD.ORG
Subject:   Re: securelevel and ipfw zero
Message-ID:  <Pine.BSF.3.96.990728162234.20420I-100000@cygnus.rush.net>
In-Reply-To: <Pine.BSF.4.10.9907281615560.92555-100000@janus.syracuse.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 28 Jul 1999, Brian F. Feldman wrote:

> 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*

and so it should remain, changes that provide readability to
code should be committed, the only time documentation of code
is wrong is when it it is incorrect.

Increasing the size of the cvs repo is not a consideration when
worthwhile docs can be incorperated, especially when the person
who needs to maintain it requires changess for readability.

Is there a point to hindering the maintainer's ongoing work?

-Alfred Perlstein - [bright@rush.net|bright@wintelcom.net] 
systems administrator and programmer
    Wintelcom - http://www.wintelcom.net/




To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-ipfw" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.3.96.990728162234.20420I-100000>