PERFORCE change 19023 for review

[Robert Watson]

On Thu, 10 Oct 2002, Chris Vance wrote:

> On Thu, 10 Oct 2002, John Baldwin wrote:
> 
> > On 10-Oct-2002 Chris Vance wrote:
> > > http://perforce.freebsd.org/p4db/chv.cgi?CH=19023
> > >
> > > Change 19023 by cvance at cvance_laptop on 2002/10/10 07:14:36
> > >
> > >       Fix bug.
> >
> > I don't know if the style below is some NAI thing, but if you were using
> > style(9), then the second lines would be at fixed offsets and wouldn't
> > have to change.  This would result in a smaller, easier-to-read diff that
> > more obviously points out that you just added a single !.  Just something
> > to think about.  There are a lot of pratical things behind style(9).
> 
> In case folks were curious, It's not a style thing so much as an issue
> with third-party code.  I'm trying to maintain compatibility as close as
> possible to the upstream (SELinux) codebase.  Obviously this particular
> chunk doesn't reduce _that_ diff, but it's the style for the surrounding
> code, so I've adopted it. 

Yeah -- the usual exception to style rules is vendor code where long term
synchronization and maintenance is an issue, and the diff reduction
concern is relative to the vendor code rather than local code.

Robert N M Watson             FreeBSD Core Team, TrustedBSD Projects
robert at fledge.watson.org      Network Associates Laboratories


To Unsubscribe: send mail to majordomo at trustedbsd.org
with "unsubscribe trustedbsd-cvs" in the body of the message