Subject: Re: [PATCH 2/2] Include unistd.h.

From: Jonathan Nieder <[email-masked]>
Date: Thu, 26 May 2011 11:39:21 -0500

Junio C Hamano wrote:

> --- a/compat/fnmatch/fnmatch.c
> +++ b/compat/fnmatch/fnmatch.c
> @@ -127,6 +127,10 @@ extern char *getenv ();
>  extern int errno;
>  # endif
> +
> +# ifndef NULL
> +#  define NULL 0
> +# endif

Makes a lot of sense.  This fits well with the style of the rest of
the file and other projects using glibc fnmatch could reuse the fix
even if targeting ancient platforms.