Re: [PATCH 0/2] Clean up disabled build warnings

2020-06-11 Thread Willy Tarreau
On Thu, Jun 11, 2020 at 03:04:39PM +0200, Tim Düsterhus wrote: > > Tim Düsterhus (2): > > BUILD: Remove nowarn for warnings that do not trigger > > BUILD: Re-enable -Wimplicit-fallthrough > > With the new -dev9 I'd like to move this thread up the INBOX again, in > case you want to resolve

Re: [PATCH 0/2] Clean up disabled build warnings

2020-06-11 Thread Tim Düsterhus
Willy, William Am 29.05.20 um 14:35 schrieb Tim Duesterhus: > These two patches re-enable some build warnings in the Makefile. The first one > only enables those that did not trigger for me at all. > > The second one re-enables -Wimplicit-fallthrough. I could not adjust all > places, because I

Re: [PATCH 0/2] Clean up disabled build warnings

2020-05-29 Thread Willy Tarreau
On Fri, May 29, 2020 at 03:43:06PM +0200, Tim Düsterhus wrote: > Willy, > > Am 29.05.20 um 14:57 schrieb Willy Tarreau: > > Hi Tim, > > > > On Fri, May 29, 2020 at 02:35:49PM +0200, Tim Duesterhus wrote: > >> Hi there, > >> > >> These two patches re-enable some build warnings in the Makefile.

Re: [PATCH 0/2] Clean up disabled build warnings

2020-05-29 Thread Tim Düsterhus
Willy, Am 29.05.20 um 14:57 schrieb Willy Tarreau: > Hi Tim, > > On Fri, May 29, 2020 at 02:35:49PM +0200, Tim Duesterhus wrote: >> Hi there, >> >> These two patches re-enable some build warnings in the Makefile. The first >> one >> only enables those that did not trigger for me at all. > >

Re: [PATCH 0/2] Clean up disabled build warnings

2020-05-29 Thread Willy Tarreau
Hi Tim, On Fri, May 29, 2020 at 02:35:49PM +0200, Tim Duesterhus wrote: > Hi there, > > These two patches re-enable some build warnings in the Makefile. The first one > only enables those that did not trigger for me at all. OK, I agree with not removing warnings that don't trigger. > The

[PATCH 0/2] Clean up disabled build warnings

2020-05-29 Thread Tim Duesterhus
Hi there, These two patches re-enable some build warnings in the Makefile. The first one only enables those that did not trigger for me at all. The second one re-enables -Wimplicit-fallthrough. I could not adjust all places, because I was not sure whether they work as intended. I've Cc't Willy