[PATCH] Cygwin: define byteswap.h inlines as macros

Yaakov Selkowitz yselkowitz@cygwin.com
Tue Mar 15 15:23:00 GMT 2016


On 2016-03-15 06:46, Corinna Vinschen wrote:
> On Mar 15 11:55, Václav Haisman wrote:
>> Would it not be better to leave the original functions as they were
>> and simply use these defines?
>>
>> #define bswap_16 bswap_16
>> #define bswap_32 bswap_32
>> #define bswap_64 bswap_64
>>
>> I believe this is valid C and C++. Untested.

Already tried that, it leads to compiling issues at least with the 
package that triggered this patch in the first place.

-- 
Yaakov



More information about the Cygwin-patches mailing list