From: | Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> |
---|---|
To: | Noah Misch <noah(at)leadboat(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Synchronize with imath upstream |
Date: | 2019-02-03 07:07:36 |
Message-ID: | 87tvhl75aw.fsf@news-spur.riddles.org.uk |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
>>>>> "Noah" == Noah Misch <noah(at)leadboat(dot)com> writes:
>> I found it much simpler to strip out -Wdeclaration-after-statement
>> instead:
>>
>> $(RYU_OBJS): override CFLAGS := $(filter-out -Wdeclaration-after-statement,$(CFLAGS))
Noah> The -Wno-declaration-after-statement approach takes eight lines
Noah> of code, and the filter-out approach takes one. On the other
Noah> hand, using $(filter-out) changes any runs of whitespace to
Noah> single spaces ("$(filter-out foo,a b c)" yields "a b c"). We do
Noah> risk that with CPPFLAGS and LDFLAGS in a few places. I don't want
Noah> to proliferate that practice, because it changes semantics of
Noah> CFLAGS containing -DFOO="arbitrary text".
In that case I propose that we avoid the whole issue by removing
-Wdeclaration-after-statement entirely.
So far, expressed opinions have run about 4:2 in favour of allowing
mixed declarations and code.
--
Andrew (irc:RhodiumToad)
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Gierth | 2019-02-03 07:09:53 | Re: Able to do ALTER DEFAULT PRIVILEGES from a user who is not the owner |
Previous Message | rajan | 2019-02-03 07:06:23 | Re: Able to do ALTER DEFAULT PRIVILEGES from a user who is not the owner |