Subject: Re: CVS commit: basesrc
To: Klaus Klein <kleink@netbsd.org>
From: Todd Vierling <tv@pobox.com>
List: source-changes
Date: 12/25/1999 23:10:14
: > For the benefit of G++'s null pointer constant implementation, #define NULL
: > as __null with egcs 1.0 (GCC 2.90) and above.  As several headers are affected
: > by this change, move the definition into a new header file, <null.h>, to ease
: > maintenance.

Shouldn't this be a <sys/null.h>?  It certainly shouldn't be in the root
include directory if we can help it, and kernel code is certainly
affected.  (Yes, it's perfectly possible to write a C++ kernel module, as
long as an implementation of new and delete is provided....)

...But since we're on the subject, why can't we move this into a
<sys/stddef.h> and make system headers include this (and user headers
include <stddef.h>, which in turn includes the previous)?

-- 
-- Todd Vierling (tv@pobox.com)