This is the mail archive of the cygwin-patches mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFC][patch] cygwin/singal.h is not compatible with -std=c89 or -std=c99


On Mon, Nov 27, 2006 at 05:24:14PM +0100, Corinna Vinschen wrote:
>On Nov 27 11:16, Christopher Faylor wrote:
>> On Mon, Nov 27, 2006 at 04:35:19PM +0100, Corinna Vinschen wrote:
>> >On Nov 27 10:17, Christopher Faylor wrote:
>> >> How about the alternative "Don't do that" approach?  I think there are
>> >> other parts of the header files which won't work with -std=c89.  I've
>> >> always been coding with the understanding that this is a GNU C environment.
>> >
>> >Well, BSD and Linux are using the more portable approach.  Why should
>> >Cygwin stand back?
>> 
>> Because the "portable" approach pollutes the namespace and, if we change
>> this, there are several other places which also need to be changed.
>> 
>> Anonymous unions are a feature of gcc and MSVC.  If we really have to do
>> this then I would like to adopt the convention of  using __extension__.
>
>Ok, no problem.  I don't like this __extension__ stuff much but I can
>live with it.

Ok.  I'll check in a patch which adds __extension__ to various places in
include/cygwin/signal.h.

cgf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]