dblink: Small code rearrangement for clarity
suggested by Tom Lane
This commit is contained in:
parent
afd79873a0
commit
e59b74a3fc
@ -360,7 +360,7 @@ Datum
|
|||||||
dblink_open(PG_FUNCTION_ARGS)
|
dblink_open(PG_FUNCTION_ARGS)
|
||||||
{
|
{
|
||||||
PGresult *res = NULL;
|
PGresult *res = NULL;
|
||||||
PGconn *conn = NULL;
|
PGconn *conn;
|
||||||
char *curname = NULL;
|
char *curname = NULL;
|
||||||
char *sql = NULL;
|
char *sql = NULL;
|
||||||
char *conname = NULL;
|
char *conname = NULL;
|
||||||
@ -408,7 +408,7 @@ dblink_open(PG_FUNCTION_ARGS)
|
|||||||
|
|
||||||
if (!rconn || !rconn->conn)
|
if (!rconn || !rconn->conn)
|
||||||
dblink_conn_not_avail(conname);
|
dblink_conn_not_avail(conname);
|
||||||
else
|
|
||||||
conn = rconn->conn;
|
conn = rconn->conn;
|
||||||
|
|
||||||
/* If we are not in a transaction, start one */
|
/* If we are not in a transaction, start one */
|
||||||
@ -451,7 +451,7 @@ PG_FUNCTION_INFO_V1(dblink_close);
|
|||||||
Datum
|
Datum
|
||||||
dblink_close(PG_FUNCTION_ARGS)
|
dblink_close(PG_FUNCTION_ARGS)
|
||||||
{
|
{
|
||||||
PGconn *conn = NULL;
|
PGconn *conn;
|
||||||
PGresult *res = NULL;
|
PGresult *res = NULL;
|
||||||
char *curname = NULL;
|
char *curname = NULL;
|
||||||
char *conname = NULL;
|
char *conname = NULL;
|
||||||
@ -495,7 +495,7 @@ dblink_close(PG_FUNCTION_ARGS)
|
|||||||
|
|
||||||
if (!rconn || !rconn->conn)
|
if (!rconn || !rconn->conn)
|
||||||
dblink_conn_not_avail(conname);
|
dblink_conn_not_avail(conname);
|
||||||
else
|
|
||||||
conn = rconn->conn;
|
conn = rconn->conn;
|
||||||
|
|
||||||
appendStringInfo(&buf, "CLOSE %s", curname);
|
appendStringInfo(&buf, "CLOSE %s", curname);
|
||||||
|
Loading…
x
Reference in New Issue
Block a user