Re: [Qemu-devel] [PATCH] configure: include stddef.h for NULL

[Scott Wood <scottwood@freescale.com>]

On Tue, 5 Oct 2010 10:03:35 +0200
Markus Armbruster <armbru@redhat.com> wrote:

> Scott Wood <scottwood@freescale.com> writes:
> 
> > This fixes an observed failure to detect madvise() on Linux.
> >
> > To avoid similar issues, all other tests that use NULL but don't already
> > have stddef.h are also fixed.
> 
> Appreciated, but several instances don't need fixing, because existing
> includes already provide NULL.
> 
> > Signed-off-by: Scott Wood <scottwood@freescale.com>
> > ---
> >  configure |    9 +++++++++
> >  1 files changed, 9 insertions(+), 0 deletions(-)
> >
> > diff --git a/configure b/configure
> > index e0d34fd..9c4f1db 100755
> > --- a/configure
> > +++ b/configure
> > @@ -1276,6 +1276,7 @@ if test "$vnc_sasl" != "no" ; then
> >    cat > $TMPC <<EOF
> >  #include <sasl/sasl.h>
> >  #include <stdio.h>
> > +#include <stddef.h>
> 
> stdio.h provides NULL.

Hmm, I had looked up a reference to see if stdio.h is defined to
supply this, and didn't see it.  Today I see it. :-P

I'll respin the patch.

-Scott