From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [COMMITTERS] pgsql: Allow Pin/UnpinBuffer to operate in a lockfree manner. |
Date: | 2016-04-15 02:17:02 |
Message-ID: | 20160415021702.rw6n725aqkfluf5w@alap3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On 2016-04-13 23:31:21 -0700, Andres Freund wrote:
> I'm also putting up an animal with clang that uses
> CFLAGS='-std=c89 -Wc99-extensions -Werror=c99-extensions'
> which actually catches this.
Hm. Doing so I found the following in 9.3:
/home/andres/src/postgresql-9.3/src/bin/pg_dump/parallel.c:561:23: error: initializer for aggregate is not a compile-time constant [-Werror,-Wc99-extensions]
int pipefd[2] = {pipeMW[PIPE_READ], pipeWM[PIPE_WRITE]};
^~~~~~~~~~~~~~~~~
which is, afaics, the same class of issue we're hitting on master right
now. I apparently fixed that, via Robert, back in 59202fae0. I'd planned
to put up that animal (mylodon) for all branches, but given that that's
been unfixed for years I'm not sure. What do you think?
- Andres
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2016-04-15 02:28:41 | pgsql: Make init_spin_delay() C89 compliant #2. |
Previous Message | Tom Lane | 2016-04-14 23:42:56 | pgsql: Fix core dump in ReorderBufferRestoreChange on alignment-picky p |
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2016-04-15 02:35:19 | Re: Disallow unique index on system columns |
Previous Message | Tom Lane | 2016-04-15 01:56:13 | Re: Disallow unique index on system columns |