[PATCH] Fix "/*" within comment warning

Yaakov Selkowitz yselkowi@redhat.com
Mon Mar 21 14:11:00 GMT 2016


On 2016-03-21 06:44, Corinna Vinschen wrote:
>> +/* These types are required by netinet/ *.h on Cygwin */
>>   #if __BSD_VISIBLE || defined(__CYGWIN__)
>>   #ifndef _BSDTYPES_DEFINED
>>   /* also defined in mingw/gmon.h and in w32api/winsock[2].h */
>
> Apart from the comment patch, do we really need "|| defined(__CYGWIN__)"
> here?

The comment explains the very reason why those types are always needed.

-- 
Yaakov Selkowitz
Associate Software Engineer, ARM
Red Hat, Inc.



More information about the Newlib mailing list