Re: [Mingw-w64-public] [PATCH] uianimation.h: Add missed defines

2018-02-27 Thread Ruslan Garipov
Hi, Jacek! Thanks a lot for your help: I had to use `const`ants and not `define`s, that was my error. Since my patch for the header file was previously marked as "ok", I will push my patch (for 'uianimation.h' only) without waiting for your patch (for 'uianimation.idl'). (I executed `widl` e

Re: [Mingw-w64-public] [PATCH] uianimation.h: Add missed defines

2018-02-27 Thread Ruslan Garipov
Hi, Jacek! It looks like I don't understand something with IDL or 'widl', but if I add those `define`s into the IDL file they do **not** appear in the result header file (i.e. "result file" of the 'widl'). Can you help me with that? On February 19, 2018 2:07:03 AM Jacek Caban wrote: Hi Rus

Re: [Mingw-w64-public] [PATCH] uianimation.h: Add missed defines

2018-02-27 Thread Jacek Caban
Hi Ruslan, On 2/17/18 5:45 PM, Ruslan Garipov wrote: diff --git a/mingw-w64-headers/include/uianimation.h b/mingw-w64-headers/include/uianimation.h index 5fce04b9..7893ba56 100644 --- a/mingw-w64-headers/include/uianimation.h +++ b/mingw-w64-headers/include/uianimation.h Please fix the IDL f

Re: [Mingw-w64-public] [PATCH] uianimation.h: Add missed defines

2018-02-27 Thread Jacek Caban
Hi Ruslan, Those are actually constants in IDL file that widl translates to defined. I sent a patch to IDL file. Headers are not generated in default build, they are stored in git instead. To generate them, you need to have widl available and configure mingw-w64 with --with-widl option. Th

[Mingw-w64-public] [PATCH] uianimation.idl: Added missing constants.

2018-02-27 Thread Jacek Caban
Based on patch by Ruslan Garipov. Signed-off-by: Jacek Caban --- mingw-w64-headers/include/uianimation.idl | 6 ++ 1 file changed, 6 insertions(+) diff --git a/mingw-w64-headers/include/uianimation.idl b/mingw-w64-headers/include/uianimation.idl index e010887f..3c559aef 100644 --- a/ming

Re: [Mingw-w64-public] Patch: work around stdcall function returning an aggregate is incompatible with MS ABI issue

2018-02-27 Thread Jacek Caban
Hi Tom, On 2/27/18 6:04 AM, Tom Ritter wrote: This patch adds wrapper functions around methods that are affected by the stdcall function returning an aggregate is incompatible with MS ABI gcc bug Background https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64384 https://sourceforge.net/p/mingw-w64/

Re: [Mingw-w64-public] [PATCH] msvcrt: Remove "DATA" attributes for _onexit for i386

2018-02-27 Thread Martin Storsjö
On Fri, 9 Feb 2018, Martell Malone wrote: LGTM. This is exactly how I was working around it also. Pushed. // Martin -- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org