Re: Synchronize with imath upstream - Mailing list pgsql-hackers

From Andrew Gierth
Subject Re: Synchronize with imath upstream
Date
Msg-id 87tvhl75aw.fsf@news-spur.riddles.org.uk
Whole thread Raw
In response to Re: Synchronize with imath upstream  (Noah Misch <noah@leadboat.com>)
Responses Re: Synchronize with imath upstream  (David Fetter <david@fetter.org>)
List pgsql-hackers
>>>>> "Noah" == Noah Misch <noah@leadboat.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)


pgsql-hackers by date:

Previous
From: rajan
Date:
Subject: Re: Able to do ALTER DEFAULT PRIVILEGES from a user who is not theowner
Next
From: Andrew Gierth
Date:
Subject: Re: Able to do ALTER DEFAULT PRIVILEGES from a user who is not the owner