pgsql: Remove dead code in ECPGconnect(), and improve documentation. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Remove dead code in ECPGconnect(), and improve documentation.
Date
Msg-id E1lAIDz-0003EC-R6@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Remove dead code in ECPGconnect(), and improve documentation.

The stanza in ECPGconnect() that intended to allow specification of a
Unix socket directory path in place of a port has never executed since
it was committed, nearly two decades ago; the preceding strrchr()
already found the last colon so there cannot be another one.  The lack
of complaints about that is doubtless related to the fact that no
user-facing documentation suggested it was possible.

Rather than try to fix that up, let's just remove the unreachable
code, and instead document the way that does work to write a socket
directory path, namely specifying it as a "host" option.

In support of that, make another pass at clarifying the syntax
documentation for ECPG connection targets, particularly documenting
which things are parsed as identifiers and where to use double quotes.
Rearrange some things that seemed poorly ordered, and fix a couple of
minor doc errors.

Kyotaro Horiguchi, per gripe from Shenhao Wang
(docs changes mostly by me)

Discussion: https://postgr.es/m/ae52a416bbbf459c96bab30b3038e06c@G08CNEXMBPEKD06.g08.fujitsu.local

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/62535cae9723afc48173ba1be65f1c7491813fc2

Modified Files
--------------
doc/src/sgml/ecpg.sgml                | 64 ++++++++++++++++++++++-------------
src/interfaces/ecpg/ecpglib/connect.c | 48 ++++++--------------------
2 files changed, 52 insertions(+), 60 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Simplify jsonfuncs.c code by using strtoint() not strtol().
Next
From: Peter Geoghegan
Date:
Subject: Re: pgsql: Add functions to 'pageinspect' to inspect GiST indexes.