svn commit: r186751 - in head/sys: netinet6 netipsec

[Steve Kargl]

On Mon, Jan 05, 2009 at 07:45:47PM +0000, Bjoern A. Zeeb wrote:
> On Mon, 5 Jan 2009, Christoph Mallon wrote:
> 
> Hi,
> 
> >Bjoern A. Zeeb schrieb:
> >>I have broken it out from a larger patch; the C99 initializer is here:
> >>http://people.freebsd.org/~bz/20090105-02-c99-initializers.diff
> >>
> >>I case someone can give it a quick glance I'll commit them.
> >
> >+	.pr_flags	= PR_ATOMIC|PR_ADDR,
> >
> >style(9) wants spaces around binary operators.
> 
> I am not sure it does. In case it does I am not going to break
> consistency with all other 65 places that don't do for pr_flags;
> but it seems I should change the <tab>=<space> to <space>=<tab>.
> Thanks for making me look, and thanks for review!
> 

>From style(9):

    Unary operators do not require spaces, binary operators do.

and

    Stylistic changes (including whitespace changes) are hard on
	the source repository and are to be avoided without good reason. 

So, it appears your change conforms to the prevailing style of the
file, and is acceptable.

-- 
Steve