Re: [Mingw-w64-public] Problems with using printf format specs with GCC 7

2017-09-11 Thread Eli Zaretskii
> From: > Date: Tue, 12 Sep 2017 10:05:12 +1000 > > Not sure what to make of the format warnings (I'll leave that to others), > but I think the post that you cited does reveal an instance of Emacs doing > something wrong wrt the redefining of _WIN32_WINNT. > This

Re: [Mingw-w64-public] Problems with using printf format specs with GCC 7

2017-09-11 Thread sisyphus1
-Original Message- From: Eli Zaretskii Sent: Tuesday, September 12, 2017 3:19 AM To: mingw-w64-public@lists.sourceforge.net Subject: [Mingw-w64-public] Problems with using printf format specs with GCC 7 Does Emacs do anything wrong in these cases? Not sure what to make of the

Re: [Mingw-w64-public] Problems with using printf format specs with GCC 7

2017-09-11 Thread JonY via Mingw-w64-public
On 09/11/2017 05:19 PM, Eli Zaretskii wrote: > Hi, > > Kindly take a look at the issues with building current development > sources of GNU Emacs with MinGW64, described here: > > http://lists.gnu.org/archive/html/emacs-devel/2017-09/msg00171.html > > The build emits a lot of warnings related

Re: [Mingw-w64-public] [PATCH 1/3] crt: Add a def file for ucrtbase.dll

2017-09-11 Thread Martin Storsjö
On Mon, 11 Sep 2017, Jacek Caban wrote: Hi Martin, On 11.09.2017 14:48, Martin Storsjö wrote: +_ctime32 +; _ctime32_s replaced by emu +_ctime64 +; _ctime64_s replaced by emu This doesn't look right, we should use .def file to expose those. I noticed a few similar other cases in the patch.

Re: [Mingw-w64-public] [PATCH] sal.h: Added __*_opt macros.

2017-09-11 Thread Martin Storsjö
On Mon, 11 Sep 2017, Jacek Caban wrote: Please review. Signed-off-by: Jacek Caban --- mingw-w64-headers/include/sal.h | 3 +++ 1 file changed, 3 insertions(+) Not familiar with this header before, but given the rest of the file, this patch seems like it'd be ok - so

[Mingw-w64-public] [PATCH] sal.h: Added __*_opt macros.

2017-09-11 Thread Jacek Caban
Please review. Signed-off-by: Jacek Caban --- mingw-w64-headers/include/sal.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/mingw-w64-headers/include/sal.h b/mingw-w64-headers/include/sal.h index 9088d5ab..0a2df8b4 100644 --- a/mingw-w64-headers/include/sal.h

Re: [Mingw-w64-public] [PATCH] setjmp: Support setjmp on arm64

2017-09-11 Thread Martin Storsjö
On Fri, 8 Sep 2017, JonY via Mingw-w64-public wrote: On 09/08/2017 06:48 AM, Martin Storsjö wrote: Signed-off-by: Martin Storsjö --- mingw-w64-crt/misc/mingw_getsp.S | 5 - mingw-w64-headers/crt/setjmp.h | 26 ++ 2 files changed, 30

Re: [Mingw-w64-public] [PATCH] setjmp: Only use _setjmp3 on i386

2017-09-11 Thread Martin Storsjö
On Thu, 7 Sep 2017, Martin Storsjö wrote: Don't use _WIN64 to decide this, but check explicitly for i386, and use _setjmp on other platforms. This fixes compilation of code using setjmp on arm32. Signed-off-by: Martin Storsjö --- mingw-w64-headers/crt/setjmp.h | 6 +++--- 1

Re: [Mingw-w64-public] [PATCH 2/2] crt: Correct aliases in newdev.def

2017-09-11 Thread Martin Storsjö
On Thu, 7 Sep 2017, Martin Storsjö wrote: To actually produce the right kind of intended alias, use double equals characters as in all other places. This has been the same since it was initially added in 9ff1fefdadd4dec5826ff653cbac6f263bf47a72 though. Signed-off-by: Martin Storsjö