Re: [Mingw-w64-public] [PATCH] for push/pop macro problem

2016-08-21 Thread David Wohlferd
On 8/21/2016 11:27 PM, Martin Storsjö wrote: > On Sun, 21 Aug 2016, David Wohlferd wrote: > >> Under certain circumstances, the #pragma pop_macro("__has_builtin") at the >> bottom of intrin-impl.h can be called without ever having hit the #pragma >> push_macro("__has_builtin") at the top. Clang wa

Re: [Mingw-w64-public] [PATCH] for missing voids

2016-08-21 Thread David Wohlferd
On 8/21/2016 11:17 PM, Martin Storsjö wrote: > On Sun, 21 Aug 2016, David Wohlferd wrote: > >> To my surprise, these two statements have (slightly) different meanings: >> >> STDAPI MFUnregisterPlatformFromMMCSS (); >> STDAPI MFUnregisterPlatformFromMMCSS (void); >> >> And clang complains about

Re: [Mingw-w64-public] [PATCH] for missing voids

2016-08-21 Thread lhmouse
GCC (but not G++) doesn't enable this warning unless requested with `-Wstrict-prototypes` explicitly. -- Best regards, lh_mouse 2016-08-22 - 发件人:David Wohlferd 发送日期:2016-08-22 11:20 收件人:mi

Re: [Mingw-w64-public] [PATCH] for push/pop macro problem

2016-08-21 Thread Martin Storsjö
On Sun, 21 Aug 2016, David Wohlferd wrote: > Under certain circumstances, the #pragma pop_macro("__has_builtin") at the > bottom of intrin-impl.h can be called without ever having hit the #pragma > push_macro("__has_builtin") at the top. Clang warns about this, so I have > moved the push appro

Re: [Mingw-w64-public] [PATCH] variadic functions can't be stdcall

2016-08-21 Thread Martin Storsjö
On Sun, 21 Aug 2016, David Wohlferd wrote: > By definition, functions with variable numbers of parameters cannot be > stdcall. Clang complains (warning: stdcall calling convention ignored on > variadic function). > > Attached. Seems ok to me. I assume GCC did the same, ignored the attribute si

Re: [Mingw-w64-public] [PATCH] for missing voids

2016-08-21 Thread Martin Storsjö
On Sun, 21 Aug 2016, David Wohlferd wrote: > To my surprise, these two statements have (slightly) different meanings: > > STDAPI MFUnregisterPlatformFromMMCSS (); > STDAPI MFUnregisterPlatformFromMMCSS (void); > > And clang complains about it (warning: function with no prototype cannot use > th

Re: [Mingw-w64-public] [PATCH] for sinhl()

2016-08-21 Thread Martin Storsjö
On Sun, 21 Aug 2016, David Wohlferd wrote: > In this function: > >long double sinhl(long double x) > > there is a call to fabs: > > (fabs (x) > (MAXLOGL + LOGE2L))) > > However, fabs doesn't take a (long double), it only takes a (double). Clang > complains about the truncation (warning:

[Mingw-w64-public] [PATCH] for push/pop macro problem

2016-08-21 Thread David Wohlferd
Under certain circumstances, the #pragma pop_macro("__has_builtin") at the bottom of intrin-impl.h can be called without ever having hit the #pragma push_macro("__has_builtin") at the top. Clang warns about this, so I have moved the push appropriately. Patch attached. dw diff --git a/mingw-

[Mingw-w64-public] [PATCH] variadic functions can't be stdcall

2016-08-21 Thread David Wohlferd
By definition, functions with variable numbers of parameters cannot be stdcall. Clang complains (warning: stdcall calling convention ignored on variadic function). Attached. dw diff --git a/mingw-w64-headers/include/strsafe.h b/mingw-w64-headers/include/strsafe.h index 8d26b6a..36c7796 10064

[Mingw-w64-public] [PATCH] for missing voids

2016-08-21 Thread David Wohlferd
To my surprise, these two statements have (slightly) different meanings: STDAPI MFUnregisterPlatformFromMMCSS (); STDAPI MFUnregisterPlatformFromMMCSS (void); And clang complains about it (warning: function with no prototype cannot use the stdcall calling convention). I have added 'void'

[Mingw-w64-public] [PATCH] for sinhl()

2016-08-21 Thread David Wohlferd
In this function: long double sinhl(long double x) there is a call to fabs: (fabs (x) > (MAXLOGL + LOGE2L))) However, fabs doesn't take a (long double), it only takes a (double). Clang complains about the truncation (warning: absolute value function 'fabs' given an argument of typ

Re: [Mingw-w64-public] error: operator '==' has no left operand, #if (defined(_FILE_OFFSET_BITS) && (_FILE_OFFSET_BITS == 64))

2016-08-21 Thread David Wohlferd
To get this error, I have to have a line like: #define _FILE_OFFSET_BITS _FILE_OFFSET_BITS should either be undefined, or be set to 32 or 64 (see https://www.gnu.org/software/libc/manual/html_node/Feature-Test-Macros.html). 'Blank' is not a valid setting. I'm not sure where this is happening i

[Mingw-w64-public] error: operator '==' has no left operand, #if (defined(_FILE_OFFSET_BITS) && (_FILE_OFFSET_BITS == 64))

2016-08-21 Thread kl222
When I compile tiff, the following error occurred: [ 4%] Building C object libtiff/CMakeFiles/tiff.dir/tif_close.c.obj In file included from D:/msys32/mingw32/i686-w64-mingw32/include/io.h:169:0, from D:/msys32/mingw32/i686-w64-mingw32/include/fcntl.h:8, from D