From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> |
Cc: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: Fix integer-overflow problems in interval comparison. |
Date: | 2017-04-17 17:04:31 |
Message-ID: | 7775.1492448671@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> writes:
> On 4/5/17 23:51, Tom Lane wrote:
>> Fix integer-overflow problems in interval comparison.
> This is failing cpluspluscheck now because C++ does not have
> _Static_assert, which is used in int128.h.
Hmm. We could drop that assert, or move it to some .c file, or
wrap it in "#ifndef __cplusplus" ... but really it seems like a
more generic solution would be appropriate. This won't be the
last time somebody tries to do that, what with our increasing
use of inline functions.
Maybe the definition of StaticAssertStmt should be tweaked based on
__cplusplus? Or maybe the problem is cpluspluscheck's test methodology,
ie you ought to run configure with CC=c++ before trying to compile the
headers?
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2017-04-17 17:07:44 | Re: pgsql: Fix integer-overflow problems in interval comparison. |
Previous Message | Alvaro Herrera | 2017-04-17 17:03:03 | pgsql: Fix extended statistics with partial analyzes |