[PATCH] --std=c89 error in sys/signal.h

Yaakov (Cygwin/X) yselkowitz@users.sourceforge.net
Mon Oct 5 01:44:00 GMT 2009


On 29/09/2009 20:11, Yaakov (Cygwin/X) wrote:
> On 29/09/2009 19:35, Yaakov (Cygwin/X) wrote:
>> Anyway, to answer the question, AFAICS in glibc, <signal.h> #include
>> <bits/types.h> unconditionally[1]. (<sys/signal.h> is just one line:
>> #include <signal.h> [2])
>>
>> So should I take the first route, patching newlib instead?
>
> OTOH, this comment in the offending hunk of <sys/signal.h> (which is
> only for Cygwin and RTEMS) makes me wonder:
>
> /* The first argument to kill should be pid_t. Right now
> <sys/types.h> always defines pid_t to be int. If that ever
> changes, then we will need to do something else, perhaps along the
> lines of <machine/types.h>. */
> int _EXFUN(kill, (int, int));
> int _EXFUN(killpg, (pid_t, int));
>
> Is that supposed to mean that we don't want to use pid_t here at all,
> and the intended solution would be to change killpg to (int, int), as
> ugly as that is, leaving only <cygwin/signal.h> needing the #include
> <sys/types.h>?

Ping?


Yaakov



More information about the Cygwin-patches mailing list