Re: [Mingw-w64-public] [PATCH] Fix gendef fallthroughts and enable winpthreads library build

2017-10-06 Thread JonY via Mingw-w64-public
On 10/06/2017 05:40 PM, Ozkan Sezer wrote: > On 10/6/17, JonY via Mingw-w64-public > wrote: >> Patches OK? > > gendef fallthrough patch looks bogus to me. > Should it follow the patterns more strictly? https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/ signature.asc

Re: [Mingw-w64-public] [PATCH] Fix gendef fallthroughts and enable winpthreads library build

2017-10-06 Thread Ozkan Sezer
On 10/6/17, JonY via Mingw-w64-public wrote: > Patches OK? gendef fallthrough patch looks bogus to me. -- Check out the vibrant tech community on one of the world's most engaging tech sites, Slashdot.org! http://sdm.link

Re: [Mingw-w64-public] [PATCH] Fix gendef fallthroughts and enable winpthreads library build

2017-10-06 Thread Kai Tietz via Mingw-w64-public
Hello JonY, the winpthread patch is ok. But I have to agree that this "fallthrough feature" of gcc is more or less pretty bad ... Regards, Kai 2017-10-06 15:52 GMT+02:00 JonY via Mingw-w64-public : > Patches OK? > > > -

Re: [Mingw-w64-public] [PATCH] Fix gendef fallthroughts and enable winpthreads library build

2017-10-06 Thread Martell Malone
The winpthreads patch LGTM. Will need someone else to approve/comment on the gendef patch. On Fri 6 Oct 2017 at 06:54, JonY via Mingw-w64-public < mingw-w64-public@lists.sourceforge.net> wrote: > Patches OK? > > > -- > Ch

[Mingw-w64-public] [PATCH] Fix gendef fallthroughts and enable winpthreads library build

2017-10-06 Thread JonY via Mingw-w64-public
Patches OK? From 8f28e1c607c0acf90ebbf37e1169c5099dc732b0 Mon Sep 17 00:00:00 2001 From: Jonathan Yong <10wa...@gmail.com> Date: Fri, 6 Oct 2017 13:36:55 + Subject: [PATCH 1/3] gendef: Fix remaining fallthrough cases Based on a patch by Marty Plummer Signed-off-by: Jonathan Yong <10wa...@gma

Re: [Mingw-w64-public] problem with multiple defined symbols in uuid and a patch

2017-10-06 Thread JonY via Mingw-w64-public
On 10/06/2017 10:07 AM, ralph engels wrote: > > Causes problems with wxwidgets so here is a proposed patch. > > Subject: [PATCH] uuid.c: Removed DEFINE_GUID also in extras-uuid.c. Patch OK after putting the list of duplicates removed in the commit message. Thanks. signature.asc Description:

[Mingw-w64-public] problem with multiple defined symbols in uuid and a patch

2017-10-06 Thread ralph engels
Causes problems with wxwidgets so here is a proposed patch. Subject: [PATCH] uuid.c: Removed DEFINE_GUID also in extras-uuid.c. ---  mingw-w64-crt/libsrc/uuid.c | 12  1 file changed, 12 deletions(-) diff --git a/mingw-w64-crt/libsrc/uuid.c b/mingw-w64-crt/libsrc/uuid.c index 481e67