On Tue, 16 Aug 2016, dw wrote: > On 8/16/2016 4:16 AM, Jacek Caban wrote: >> On 16.08.2016 12:50, dw wrote: >>> --- a/mingw-w64-headers/include/mfidl.h >>> +++ b/mingw-w64-headers/include/mfidl.h >> This file is auto generated. You should not touch it, please change .idl >> file instead. > > Strictly speaking, I believe the (duplicate) MF_xx_BYTE_ALIGNMENT defines > should be removed from mfidl.idl. But for our purposes today, I can achieve > the desired result by just changing mfapi.h (attached as defines3.patch). > Better? > > And since you mentioned it, I checked the other files and dxgi.h is also a > generated file. I have broken it out to a separate patch (dxgi.patch). > > ========= dxgi.patch ============ > Avoid 'redefine' warnings from duplicated (but microscopically different) > defines. > > ========= defines3.patch ============ > Fix minor variations in definitions that trigger compiler warnings. > > gs_support.c: > - The minor variation causes a 'redefines' error. > cephes_emath.h: > - The minor variations cause 'redefines' errors. > aviriff.h, basetyps.h, combaseapi.h, gpedit.h: > - Redefine errors. > mfapi.h: > - Redefine errors. > ntdef.h: > - Redefine errors. > winnt.h: > - Redefine errors.
Both patches ok with me, but please wait for someone else's opinion as well. // Martin ------------------------------------------------------------------------------ _______________________________________________ Mingw-w64-public mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/mingw-w64-public
