From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Fabrice Popineau Newsgroups: gmane.emacs.devel Subject: Re: Suspicious warning in W64 build Date: Tue, 12 Sep 2017 20:01:58 +0200 Message-ID: References: <1017454172.910810.1504618695244@mail.libero.it> <83tw0ezgi1.fsf@gnu.org> <1146932094.13821.1504798935795@mail.libero.it> <83mv66z66p.fsf@gnu.org> <102a4c21-c275-c73e-ec53-0d85975dc968@cs.ucla.edu> <83a825znuf.fsf@gnu.org> <831snhzkgw.fsf@gnu.org> <837ex9x7vf.fsf@gnu.org> <83tw0cwcle.fsf@gnu.org> <83h8wcw3td.fsf@gnu.org> <83377vx3d0.fsf@gnu.org> <83ingnq01t.fsf@gnu.org> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="f403045d64f01c6c44055901d817" X-Trace: blaine.gmane.org 1505337800 11769 195.159.176.226 (13 Sep 2017 21:23:20 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Wed, 13 Sep 2017 21:23:20 +0000 (UTC) Cc: Richard Copley , Emacs developers To: Eli Zaretskii Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Sep 12 20:03:22 2017 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1drpWe-0001Wx-5i for ged-emacs-devel@m.gmane.org; Tue, 12 Sep 2017 20:03:08 +0200 Original-Received: from localhost ([::1]:37890 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1drpWl-0005Va-9N for ged-emacs-devel@m.gmane.org; Tue, 12 Sep 2017 14:03:15 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:38130) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1drpW2-0005PZ-59 for emacs-devel@gnu.org; Tue, 12 Sep 2017 14:02:32 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1drpW0-0000rL-1g for emacs-devel@gnu.org; Tue, 12 Sep 2017 14:02:30 -0400 Original-Received: from smtp2.supelec.fr ([160.228.120.31]:49288) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1drpVw-0000q5-1e; Tue, 12 Sep 2017 14:02:24 -0400 Original-Received: from localhost (localhost [127.0.0.1]) by smtp2.supelec.fr (Postfix) with ESMTP id 70F5980513; Tue, 12 Sep 2017 20:02:21 +0200 (CEST) X-Virus-Scanned: amavisd-new at smtp2.supelec.fr Original-Received: from smtp2.supelec.fr ([127.0.0.1]) by localhost (smtp2.supelec.fr [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ZpqMVF-ifk-L; Tue, 12 Sep 2017 20:02:20 +0200 (CEST) Original-Received: from mail-qt0-f171.google.com (mail-qt0-f171.google.com [209.85.216.171]) by smtp2.supelec.fr (Postfix) with ESMTPSA id 4EC0580514; Tue, 12 Sep 2017 20:02:20 +0200 (CEST) Original-Received: by mail-qt0-f171.google.com with SMTP id s18so25504300qta.3; Tue, 12 Sep 2017 11:02:20 -0700 (PDT) X-Gm-Message-State: AHPjjUji8Nr2wmFmOqs6e4MpQQlt8fgz1reIhf5PoIH4wumKb1doxs7o b8Kvkizln7ZvcqplFPcq/sqcYU26JA== X-Google-Smtp-Source: AOwi7QDCmrBTBQCKZz+xGpYXysTqeXAoe/BjN8eVQcqrQUUJKN6E8jHE3cDdZyOIWAAfD1TuXYmsuWHWetvyYTMzO7o= X-Received: by 10.200.24.168 with SMTP id s37mr16903003qtj.287.1505239339125; Tue, 12 Sep 2017 11:02:19 -0700 (PDT) Original-Received: by 10.140.82.21 with HTTP; Tue, 12 Sep 2017 11:01:58 -0700 (PDT) In-Reply-To: <83ingnq01t.fsf@gnu.org> X-Gmail-Original-Message-ID: X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 160.228.120.31 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:218228 --f403045d64f01c6c44055901d817 Content-Type: text/plain; charset="UTF-8" 2017-09-12 19:49 GMT+02:00 Eli Zaretskii : > > Date: Sat, 09 Sep 2017 19:07:23 +0300 > > From: Eli Zaretskii > > Cc: angelo.g0@libero.it, emacs-devel@gnu.org > > > > > 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 --f403045d64f01c6c44055901d817 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable


2017-09-12 19:49 GMT+02:00 Eli Zaretskii <eliz@gnu.org>:
=
> Date: Sat, 09 Sep 20= 17 19:07:23 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: angelo.g0@libero.it, emacs-devel@gnu.org
>
> > At least some of the "-Wformat=3D" warnings are mislead= ing.
> > 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.=C2=A0 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 ne= ed
your help, Richard.=C2=A0 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?=C2=A0 They need to be produced using the exact GCC switches used when these files are
compiled as part of the build.=C2=A0 Let me know if you need instructions.<= br>

As an alternative, I can provide them.= =C2=A0
I have the same install with the same warnings.
=
Should you want them, they are at this address :
<= a href=3D"https://drive.google.com/file/d/0BzJyP_aI_ouHOWR6YXZ1LTRiam8/view= ?usp=3Dsharing">https://drive.google.com/file/d/0BzJyP_aI_ouHOWR6YXZ1LTRiam= 8/view?usp=3Dsharing

Compilation flags :

gcc =C2=A0-E -mtune=3Dgeneric =C2=A0-I/mingw64/incl= ude -DUSE_CRT_DLL=3D1 -I /d/source/emacs/build-master/../emacs/nt/inc -Dema= cs =C2=A0-I. -I../../emacs/src -I../lib -I../../emacs/lib =C2=A0-mtune=3Dge= neric =C2=A0 =C2=A0-pthread -mms-bitfields -isystem C:/MSys64/mingw64/inclu= de/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/incl= ude -isystem C:/MSys64/mingw64/include/freetype2 -isystem C:/MSys64/mingw64= /include/libpng16 -isystem C:/MSys64/mingw64/include/harfbuzz -isystem C:/M= Sys64/mingw64/include/glib-2.0 -isystem C:/MSys64/mingw64/lib/glib-2.0/incl= ude -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/include/f= reetype2 -isystem C:/MSys64/mingw64/include -isystem C:/MSys64/mingw64/incl= ude/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 -fopen= mp -DMAGICKCORE_HDRI_ENABLE=3D1 -DMAGICKCORE_QUANTUM_DEPTH=3D16 -D_DLL -D_M= T -fopenmp -DMAGICKCORE_HDRI_ENABLE=3D1 -DMAGICKCORE_QUANTUM_DEPTH=3D16 -D_= DLL -D_MT -isystem C:/MSys64/mingw64/include/ImageMagick-7 =C2=A0-isystem C= :/MSys64/mingw64/include/libxml2 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0-= MMD -MF deps/print.d -MP =C2=A0-isystem C:/MSys64/mingw64/include -isystem = C:/MSys64/mingw64/include/p11-kit-1 =C2=A0 =C2=A0-fno-common -W -Wabi -Wadd= ress -Waggressive-loop-optimizations -Wall -Wattributes -Wbool-compare -Wbo= ol-operation -Wbuiltin-declaration-mismatch -Wbuiltin-macro-redefined -Wcas= t-align -Wchar-subscripts -Wchkp -Wclobbered -Wcomment -Wcomments -Wcoverag= e-mismatch -Wcpp -Wdangling-else -Wdate-time -Wdeprecated -Wdeprecated-decl= arations -Wdesignated-init -Wdisabled-optimization -Wdiscarded-array-qualif= iers -Wdiscarded-qualifiers -Wdiv-by-zero -Wdouble-promotion -Wduplicated-c= ond -Wduplicate-decl-specifier -Wempty-body -Wendif-labels -Wenum-compare -= Wexpansion-to-defined -Wextra -Wformat-contains-nul -Wformat-extra-args -Wf= ormat-security -Wformat-signedness -Wformat-y2k -Wformat-zero-length -Wfram= e-address -Wfree-nonheap-object -Whsa -Wignored-attributes -Wignored-qualif= iers -Wimplicit -Wimplicit-function-declaration -Wimplicit-int -Wincompatib= le-pointer-types -Winit-self -Wint-conversion -Wint-in-bool-context -Wint-t= o-pointer-cast -Winvalid-memory-model -Winvalid-pch -Wjump-misses-init -Wlo= gical-not-parentheses -Wlogical-op -Wmain -Wmaybe-uninitialized -Wmemset-el= t-size -Wmemset-transposed-args -Wmisleading-indentation -Wmissing-braces -= Wmissing-declarations -Wmissing-include-dirs -Wmissing-parameter-type -Wmis= sing-prototypes -Wmultichar -Wnarrowing -Wnested-externs -Wnonnull -Wnonnul= l-compare -Wnull-dereference -Wodr -Wold-style-declaration -Wold-style-defi= nition -Wopenmp-simd -Woverflow -Wpacked -Wpacked-bitfield-compat -Wparenth= eses -Wpointer-arith -Wpointer-compare -Wpointer-to-int-cast -Wpragmas -Wps= abi -Wrestrict -Wreturn-local-addr -Wreturn-type -Wscalar-storage-order -Ws= equence-point -Wshift-count-negative -Wshift-count-overflow -Wshift-negativ= e-value -Wsizeof-array-argument -Wsizeof-pointer-memaccess -Wstrict-aliasin= g -Wstrict-prototypes -Wsuggest-attribute=3Dnoreturn -Wsuggest-final-method= s -Wsuggest-final-types -Wswitch-bool -Wswitch-unreachable -Wtautological-c= ompare -Wtrampolines -Wtrigraphs -Wuninitialized -Wunknown-pragmas -Wunused= -Wunused-but-set-parameter -Wunused-but-set-variable -Wunused-function -Wu= nused-label -Wunused-local-typedefs -Wunused-macros -Wunused-result -Wunuse= d-value -Wunused-variable -Wvarargs -Wvariadic-macros -Wvector-operation-pe= rformance -Wvolatile-register-var -Wwrite-strings -Walloc-size-larger-than= =3D2147483647 -Warray-bounds=3D2 -Wformat-truncation=3D2 -Wimplicit-fallthr= ough=3D5 -Wnormalized=3Dnfc -Wshift-overflow=3D2 -Wstringop-overflow=3D2 -W= vla-larger-than=3D4031 -Wno-missing-field-initializers -Wno-override-init -= Wno-sign-compare -Wno-type-limits -Wno-unused-parameter -Wno-format-nonlite= ral -Wno-pointer-sign -I/mingw64/include -O3 -g0 -mtune=3Dcorei7 -fomit-fra= me-pointer =C2=A0../../emacs/src/frame.c

Fabri= ce

--f403045d64f01c6c44055901d817--