[patch] gcc4 fixes

Robert Collins rbcollins@cygwin.com
Wed May 18 00:13:00 GMT 2005


On Tue, 2005-05-17 at 19:31 -0400, Christopher Faylor wrote:
> On Tue, May 17, 2005 at 03:50:08PM -0700, Brian Dessent wrote:
> >
> >This is just a trivial change of argument to execl() testcases, which
> >supresses the warning 'missing sentinel in function call' in gcc4 that
> >causes the tests to fail.
> >
> >winsup/testsuite
> >2005-05-17  Brian Dessent  <brian@dessent.net>
> >
> >	* winsup.api/signal-into-win32-api.c (main): Use 'NULL' instead
> >	of '0' in argument list to avoid compiler warning with gcc4.
> >	* winsup.api/ltp/execle01.c (main): Ditto.
> >	* winsup.api/ltp/execlp01.c (main): Ditto.
> >	* winsup.api/ltp/fcntl07.c (do_exec): Ditto.
> >	* winsup.api/ltp/fcntl07B.c (do_exec): Ditto.
> 
> Go ahead and check these in but please use GNU formatting conventions,
> i.e., it's (char *) NULL, not (char *)NULL.  Actually, isn't just NULL
> sufficient?

Should be: NULL is '(void *) 0' on most compilers. 

Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20050518/1b6f7fde/attachment.sig>


More information about the Cygwin-patches mailing list