Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: basesrc



: > 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@localhost)




Home | Main Index | Thread Index | Old Index