Re: C PATCH to fix bogus warning with -Wmultistatement-macros (PR c/81364)

2017-07-21 Thread David Malcolm
On Fri, 2017-07-14 at 15:38 +0200, Marek Polacek wrote: > I think David might be able to approve this one, so CCing. It's something of a stretch for my "diagnostic messages" maintainer hat, but I've looked over these changes and they look good to me. Dave > On Tue, Jul 11, 2017 at 03:23:16PM

Re: C PATCH to fix bogus warning with -Wmultistatement-macros (PR c/81364)

2017-07-20 Thread Marek Polacek
Ping. On Fri, Jul 14, 2017 at 03:38:05PM +0200, Marek Polacek wrote: > I think David might be able to approve this one, so CCing. > > On Tue, Jul 11, 2017 at 03:23:16PM +0200, Marek Polacek wrote: > > This patch fixes a bogus -Wmultistatement-macros warning. The code didn't > > notice that what

Re: C PATCH to fix bogus warning with -Wmultistatement-macros (PR c/81364)

2017-07-14 Thread Marek Polacek
I think David might be able to approve this one, so CCing. On Tue, Jul 11, 2017 at 03:23:16PM +0200, Marek Polacek wrote: > This patch fixes a bogus -Wmultistatement-macros warning. The code didn't > notice that what came after a guard such as else was actually wrapped in { } > which is a

C PATCH to fix bogus warning with -Wmultistatement-macros (PR c/81364)

2017-07-11 Thread Marek Polacek
This patch fixes a bogus -Wmultistatement-macros warning. The code didn't notice that what came after a guard such as else was actually wrapped in { } which is a correct use. This bogus warning only triggered when the body of a conditional was coming from a different expansion than the