Fix redundant error messages in client tools

petere · petere · commit 42c088be1889 · 2020-11-07T22:44:15.000+01:00
A few client tools duplicate error messages already provided by libpq. Discussion: https://www.postgresql.org/message-id/flat/3e937641-88a1-e697-612e-99bba4b8e5e4%40enterprisedb.com
diff --git a/src/bin/pg_basebackup/streamutil.c b/src/bin/pg_basebackup/streamutil.c
@@ -201,8 +201,7 @@ GetConnection(void)
 
 	if (PQstatus(tmpconn) != CONNECTION_OK)
 	{
-		fprintf(stderr, _("%s: could not connect to server: %s"),
-				progname, PQerrorMessage(tmpconn));
+		fprintf(stderr, _("%s: %s"), progname, PQerrorMessage(tmpconn));
 		PQfinish(tmpconn);
 		free(values);
 		free(keywords);
diff --git a/src/bin/pg_rewind/libpq_fetch.c b/src/bin/pg_rewind/libpq_fetch.c
@@ -50,8 +50,7 @@ libpqConnect(const char *connstr)
 
 	conn = PQconnectdb(connstr);
 	if (PQstatus(conn) == CONNECTION_BAD)
-		pg_fatal("could not connect to server: %s",
-				 PQerrorMessage(conn));
+		pg_fatal("%s", PQerrorMessage(conn));
 
 	pg_log(PG_PROGRESS, "connected to server\n");
 

Original file line number	Diff line number	Diff line change
`@@ -201,8 +201,7 @@ GetConnection(void)`
`201`	`201`
`202`	`202`	`if (PQstatus(tmpconn) != CONNECTION_OK)`
`203`	`203`	`{`
`204`		`- fprintf(stderr, _("%s: could not connect to server: %s"),`
`205`		`- progname, PQerrorMessage(tmpconn));`
	`204`	`+ fprintf(stderr, _("%s: %s"), progname, PQerrorMessage(tmpconn));`
`206`	`205`	`PQfinish(tmpconn);`
`207`	`206`	`free(values);`
`208`	`207`	`free(keywords);`