Re: pgsql: Use FLEXIBLE_ARRAY_MEMBER in a bunch more places. - Mailing list pgsql-committers

From Andres Freund
Subject Re: pgsql: Use FLEXIBLE_ARRAY_MEMBER in a bunch more places.
Date
Msg-id 20150220161745.GE12653@awork2.anarazel.de
Whole thread Raw
In response to Re: pgsql: Use FLEXIBLE_ARRAY_MEMBER in a bunch more places.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
On 2015-02-20 10:49:32 -0500, Tom Lane wrote:
> Andres Freund <andres@2ndquadrant.com> writes:
> > This triggers a large number of warnings with my preexisting clang 3.6 settings...
>
> > In file included from /home/andres/src/postgresql/src/backend/postmaster/pgstat.c:40:
> > /home/andres/src/postgresql/src/include/catalog/pg_proc.h:61:12: warning: 'proargtypes' may not be nested in a
structdue to flexible array 
>
> That's annoying.  I saw that Sun Studio was complaining similarly, but
> I figured we could ignore it.
>
> It's weird that compiler writers have such a hard time understanding
> what actually counts as a *useful* warning, ie "you've got a flexible
> array embedded in the middle of a bigger struct".  Instead we get
> either nothing (gcc) or pedantry (this).

It's two different warnings in clang, so it's not that
bad. -Wgnu-variable-sized-type-not-at-end vs -Wflexible-array-extensions

Greetings,

Andres Freund

--
 Andres Freund                       http://www.2ndQuadrant.com/
 PostgreSQL Development, 24x7 Support, Training & Services


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgsql: Have TRUNCATE update pgstat tuple counters
Next
From: Alvaro Herrera
Date:
Subject: pgsql: Remove unnecessary and unreliable test