Date: Sun, 30 Jan 2000 12:57:21 +0000 From: Brian Somers <brian@Awfulhak.org> To: Peter Wemm <peter@FreeBSD.org> Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org, brian@hak.lan.Awfulhak.org Subject: Re: cvs commit: src/sys/net intrq.c Message-ID: <200001301257.MAA04813@hak.lan.Awfulhak.org> In-Reply-To: Message from Peter Wemm <peter@FreeBSD.org> of "Sat, 29 Jan 2000 08:13:09 PST." <200001291613.IAA84069@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
> peter 2000/01/29 08:13:09 PST
>
> Modified files:
> sys/net intrq.c
> Log:
> Fix this so LINT compiles. There is no way this could have worked if
> tested with LINT. I've put back netatm/kern_include.h and maked it
> with a fixme!, otherwise NETISR_ATM isn't defined as ATM_KERNEL isn't
> defined. Defining that exposes a whole bunch of other dependencies.. :-(
>
> Revision Changes Path
> 1.3 +6 -6 src/sys/net/intrq.c
I admit I haven't compiled LINT for this change - I originally tried
compiling with and without various options to prove that they were
not interdependent.
When it was pointed out that I'd broken the alpha (as alpha can't do
ATM at all) I added the ifdefs, but this was after everything had
become unconditional. As I never tried things with ATM defined in
again (thinking *everything* was basically unconditional), I ended up
screwing up :-( Hmm, LINT is a good idea !
That's my excuse anyway. Thanks for tidying up.
--
Brian <brian@Awfulhak.org> <brian@FreeBSD.org>
<http://www.Awfulhak.org>; <brian@OpenBSD.org>
Don't _EVER_ lose your sense of humour ! <brian@FreeBSD.org.uk>
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200001301257.MAA04813>