From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Arthur Zakirov <a(dot)zakirov(at)postgrespro(dot)ru>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com> |
Subject: | Re: fix psql \conninfo & \connect when using hostaddr |
Date: | 2018-12-14 17:17:18 |
Message-ID: | 20181214171718.rmexxl43kb2drrly@alvherre.pgsql |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 2018-Nov-09, Michael Paquier wrote:
> On Thu, Nov 08, 2018 at 12:13:31PM -0300, Alvaro Herrera wrote:
> > Right, that too. Fortunately I think compilers warn about
> > mismatching indentation nowadays, at least in some cases.
>
> I don't recall seeing a compiler warning about that, but I do recall
> coverity complaining loudly about such things.
:-)
/pgsql/source/master/src/backend/catalog/namespace.c: In function 'InitRemoveTempRelations':
/pgsql/source/master/src/backend/catalog/namespace.c:4132:2: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
if (OidIsValid(namespaceOid))
^~
/pgsql/source/master/src/backend/catalog/namespace.c:4134:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
get_namespace_oid(namespaceName, true);
^~~~~~~~~~~~~~~~~
$ gcc --version
gcc (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2018-12-14 17:18:47 | Re: Remove Deprecated Exclusive Backup Mode |
Previous Message | Robert Haas | 2018-12-14 16:48:45 | Re: removal of dangling temp tables |