cvs commit: src/sys/netinet6 frag6.c

Sam Leffler sam at errno.com
Wed Oct 22 09:04:36 PDT 2003


On Wednesday 22 October 2003 08:53 am, Hajimu UMEMOTO wrote:
> Hi,
>
> >>>>> On Wed, 22 Oct 2003 08:45:31 -0700
> >>>>> Sam Leffler <sam at errno.com> said:
>
> sam> The convention used throughout the kernel is for
>
> sam> FOO_LOCK_ASSERT()
>
> sam> to generate an assertion that the specified lock is held.  It would be
> good sam> for the IPv6 code to do likewise (you appear to be using
> FOO_LOCK_CHECK sam> instead).
>
> Yes, I know.  But, I don't want to make diffs against KAME as
> possible.  So, I didn't rename IP6Q_LOCK_CHECK to IP6Q_LOCK_ASSERT.

We got there first? :)  CHECK is not meaningful in the same way that ASSERT 
is.  It also means one cannot grep for instances of macros because of this 
difference.  I suggest that soon such differences against the KAME repository 
are going to be inconsequential and you are better to have code the conforms 
to local style and conventions than to the KAME repositoriy.

	Sam



More information about the cvs-src mailing list