From: | tgl(at)postgresql(dot)org (Tom Lane) |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | pgsql: Tweak a couple of macros in the regex code to suppress compiler |
Date: | 2010-08-02 02:29:39 |
Message-ID: | 20100802022939.D1E937541D5@cvs.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Log Message:
-----------
Tweak a couple of macros in the regex code to suppress compiler warnings
from "clang". The VERR changes make an assignment unconditional, which is
probably easier to read/understand anyway, and one can hardly argue that
it's worth shaving cycles off the case of reporting another error when
one has already been detected. The INSIST change limits where that macro
can be used, but not in a way that creates a problem for any existing call.
Modified Files:
--------------
pgsql/src/backend/regex:
regcomp.c (r1.48 -> r1.49)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/regex/regcomp.c?r1=1.48&r2=1.49)
regexec.c (r1.28 -> r1.29)
(http://anoncvs.postgresql.org/cvsweb.cgi/pgsql/src/backend/regex/regexec.c?r1=1.28&r2=1.29)
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2010-08-02 03:46:54 | pgsql: Remove ancient PL/pgsql line numbering hack. |
Previous Message | Tom Lane | 2010-08-02 01:25:02 | pgsql: Fix an ancient typo that prevented the detection of conflicting |