Re: Suspicious warning in W64 build

[Fabrice Popineau]

2017-09-12 19:49 GMT+02:00 Eli Zaretskii <address@hidden>:

> Date: Sat, 09 Sep 2017 19:07:23 +0300
> From: Eli Zaretskii <address@hidden>
> Cc: address@hidden, address@hidden
>
> > At least some of the "-Wformat=" warnings are misleading.
> > Emacs has to use MSVC's rules for format strings, but GCC
> > warns based on the C standard rules implemented in GCC.
>
> Actually, these are the most worrisome, because they seem to tell your
> MinGW headers might mismatch your GCC version.  Or maybe this is a
> general MinGW64 problem that should be solved by MinGW64 developers.

OK, I got some information from the MinGW64 developers, but now I need
your help, Richard.  I need you to produce preprocessed versions of
frame.c, keyboard.c, and print.c, and send them to me.

Do you know how to produce such preprocessed versions?  They need to
be produced using the exact GCC switches used when these files are
compiled as part of the build.  Let me know if you need instructions.

As an alternative, I can provide them. 

I have the same install with the same warnings.

Should you want them, they are at this address :

https://drive.google.com/file/d/0BzJyP_aI_ouHOWR6YXZ1LTRiam8/view?usp=sharing

Compilation flags :

gcc  -E -mtune=generic  -I/mingw64/include -DUSE_CRT_DLL=1 -I /d/source/emacs/build-master/../emacs/nt/inc -Demacs  -I. -I../../emacs/src -I../lib -I../../emacs/lib  -mtune=generic    -pthread -mms-bitfields -isystem C:/MSys64/mingw64/include/librsvg-2.0 -isystem C:/MSys64/mingw64/include/gdk-pixbuf-2.0 -isystem C:/MSys64/mingw64/include/libpng16 -isystem C:/MSys64/mingw64/include/cairo -isystem C:/MSys64/mingw64/include/pixman-1 -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/include/freetype2 -isystem C:/MSys64/mingw64/include/libpng16 -isystem C:/MSys64/mingw64/include/harfbuzz -isystem C:/MSys64/mingw64/include/glib-2.0 -isystem C:/MSys64/mingw64/lib/glib-2.0/include -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/include/freetype2 -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/include/harfbuzz -isystem C:/MSys64/mingw64/include/glib-2.0 -isystem C:/MSys64/mingw64/lib/glib-2.0/include -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/include/libpng16 -isystem C:/MSys64/mingw64/include -fopenmp -DMAGICKCORE_HDRI_ENABLE=1 -DMAGICKCORE_QUANTUM_DEPTH=16 -D_DLL -D_MT -fopenmp -DMAGICKCORE_HDRI_ENABLE=1 -DMAGICKCORE_QUANTUM_DEPTH=16 -D_DLL -D_MT -isystem C:/MSys64/mingw64/include/ImageMagick-7  -isystem C:/MSys64/mingw64/include/libxml2            -MMD -MF deps/print.d -MP  -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/include/p11-kit-1    -fno-common -W -Wabi -Waddress -Waggressive-loop-optimizations -Wall -Wattributes -Wbool-compare -Wbool-operation -Wbuiltin-declaration-mismatch -Wbuiltin-macro-redefined -Wcast-align -Wchar-subscripts -Wchkp -Wclobbered -Wcomment -Wcomments -Wcoverage-mismatch -Wcpp -Wdangling-else -Wdate-time -Wdeprecated -Wdeprecated-declarations -Wdesignated-init -Wdisabled-optimization -Wdiscarded-array-qualifiers -Wdiscarded-qualifiers -Wdiv-by-zero -Wdouble-promotion -Wduplicated-cond -Wduplicate-decl-specifier -Wempty-body -Wendif-labels -Wenum-compare -Wexpansion-to-defined -Wextra -Wformat-contains-nul -Wformat-extra-args -Wformat-security -Wformat-signedness -Wformat-y2k -Wformat-zero-length -Wframe-address -Wfree-nonheap-object -Whsa -Wignored-attributes -Wignored-qualifiers -Wimplicit -Wimplicit-function-declaration -Wimplicit-int -Wincompatible-pointer-types -Winit-self -Wint-conversion -Wint-in-bool-context -Wint-to-pointer-cast -Winvalid-memory-model -Winvalid-pch -Wjump-misses-init -Wlogical-not-parentheses -Wlogical-op -Wmain -Wmaybe-uninitialized -Wmemset-elt-size -Wmemset-transposed-args -Wmisleading-indentation -Wmissing-braces -Wmissing-declarations -Wmissing-include-dirs -Wmissing-parameter-type -Wmissing-prototypes -Wmultichar -Wnarrowing -Wnested-externs -Wnonnull -Wnonnull-compare -Wnull-dereference -Wodr -Wold-style-declaration -Wold-style-definition -Wopenmp-simd -Woverflow -Wpacked -Wpacked-bitfield-compat -Wparentheses -Wpointer-arith -Wpointer-compare -Wpointer-to-int-cast -Wpragmas -Wpsabi -Wrestrict -Wreturn-local-addr -Wreturn-type -Wscalar-storage-order -Wsequence-point -Wshift-count-negative -Wshift-count-overflow -Wshift-negative-value -Wsizeof-array-argument -Wsizeof-pointer-memaccess -Wstrict-aliasing -Wstrict-prototypes -Wsuggest-attribute=noreturn -Wsuggest-final-methods -Wsuggest-final-types -Wswitch-bool -Wswitch-unreachable -Wtautological-compare -Wtrampolines -Wtrigraphs -Wuninitialized -Wunknown-pragmas -Wunused -Wunused-but-set-parameter -Wunused-but-set-variable -Wunused-function -Wunused-label -Wunused-local-typedefs -Wunused-macros -Wunused-result -Wunused-value -Wunused-variable -Wvarargs -Wvariadic-macros -Wvector-operation-performance -Wvolatile-register-var -Wwrite-strings -Walloc-size-larger-than=2147483647 -Warray-bounds=2 -Wformat-truncation=2 -Wimplicit-fallthrough=5 -Wnormalized=nfc -Wshift-overflow=2 -Wstringop-overflow=2 -Wvla-larger-than=4031 -Wno-missing-field-initializers -Wno-override-init -Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-format-nonliteral -Wno-pointer-sign -I/mingw64/include -O3 -g0 -mtune=corei7 -fomit-frame-pointer  ../../emacs/src/frame.c

Fabrice