v11-0003-minor-coesmetic-change-in-pg_restore.c.no-cfbot

From fcb2dd75ab9dc1fd3dd72c91f55d1b7b37a70c3f Mon Sep 17 00:00:00 2001
From: jian he <jian.universality@gmail.com>
Date: Thu, 23 Jan 2025 17:02:31 +0800
Subject: [PATCH v11 3/4] minor coesmetic change in pg_restore.c

---
 src/bin/pg_dump/pg_restore.c | 39 ++++++++++++++++++------------------
 1 file changed, 20 insertions(+), 19 deletions(-)

diff --git a/src/bin/pg_dump/pg_restore.c b/src/bin/pg_dump/pg_restore.c
index 79f61395ae..a715448dce 100644
--- a/src/bin/pg_dump/pg_restore.c
+++ b/src/bin/pg_dump/pg_restore.c
@@ -77,8 +77,8 @@ simple_db_oid_list_append(SimpleDatabaseOidList *list, Oid db_oid, const char *d
 static void usage(const char *progname);
 static void read_restore_filters(const char *filename, RestoreOptions *opts);
 static bool IsFileExistsInDirectory(const char *dir, const char *filename);
-static bool restoreOneDatabase(const char *inputFileSpec,
-		RestoreOptions *opts, int numWorkers, bool append_data);
+static bool restoreOneDatabase(const char *inputFileSpec, RestoreOptions *opts,
+							   int numWorkers, bool append_data);
 static int ReadOneStatement(StringInfo inBuf, FILE *pfile);
 static int restoreAllDatabases(PGconn *conn, const char *dumpdirpath,
 	SimpleStringList db_exclude_patterns, RestoreOptions *opts, int numWorkers);
@@ -87,14 +87,15 @@ static void execute_global_sql_commands(PGconn *conn, const char *dumpdirpath,
 static int filter_dbnames_for_restore(PGconn *conn,
 		SimpleDatabaseOidList *dbname_oid_list, SimpleStringList db_exclude_patterns);
 static int get_dbname_oid_list_from_mfile(const char *dumpdirpath,
-		SimpleDatabaseOidList *dbname_oid_list);
+										  SimpleDatabaseOidList *dbname_oid_list);
 static void simple_db_oid_list_append(SimpleDatabaseOidList *list, Oid db_oid,
-		const char *dbname);
+									  const char *dbname);
 static bool is_full_pattern(PGconn *conn, const char *str, const char *ptrn);
 static void simple_string_full_list_delete(SimpleStringList *list);
 static void simple_db_oid_full_list_delete(SimpleDatabaseOidList *list);
 static void simple_db_oid_list_delete(SimpleDatabaseOidList *list,
-		SimpleDatabaseOidListCell *cell, SimpleDatabaseOidListCell *prev);
+									  SimpleDatabaseOidListCell *cell,
+									  SimpleDatabaseOidListCell *prev);
 
 int
 main(int argc, char **argv)
@@ -446,8 +447,8 @@ main(int argc, char **argv)
 		/* Plain format is not supported for pg_restore. */
 		if (opts->format == archNull)
 		{
-			pg_fatal("unrecognized archive format \"%s\"; please specify \"c\", \"d\", or \"t\"",
-					opts->formatName);
+			pg_fatal("unrecognized archive format \"%s\", one of \"c\", \"d\" or \"t\" must be specified",
+					  opts->formatName);
 		}
 	}
 
@@ -458,7 +459,7 @@ main(int argc, char **argv)
 	 * --exclude-database patterns.
 	 */
 	if (inputFileSpec != NULL &&
-			!IsFileExistsInDirectory(inputFileSpec, "toc.dat"))
+		!IsFileExistsInDirectory(inputFileSpec, "toc.dat"))
 	{
 		/* If global.dat is exist, then process it. */
 		if (IsFileExistsInDirectory(pg_strdup(inputFileSpec), "global.dat"))
@@ -470,8 +471,8 @@ main(int argc, char **argv)
 			if (opts->cparams.dbname)
 			{
 				conn = connectDatabase(opts->cparams.dbname, NULL, opts->cparams.pghost,
-						opts->cparams.pgport, opts->cparams.username, TRI_DEFAULT, false,
-						progname, NULL, NULL);
+									   opts->cparams.pgport, opts->cparams.username, TRI_DEFAULT,
+									   false, progname, NULL, NULL);
 
 				if (!conn)
 					pg_fatal("could not connect to database \"%s\"", opts->cparams.dbname);
@@ -498,8 +499,8 @@ main(int argc, char **argv)
 			simple_string_full_list_delete(&db_exclude_patterns);
 
 			return exit_code;
-		}/* end if */
-	}/* end if */
+		}
+	}
 
 	return restoreOneDatabase(inputFileSpec, opts, numWorkers, false);
 }
@@ -511,7 +512,7 @@ main(int argc, char **argv)
  */
 static bool
 restoreOneDatabase(const char *inputFileSpec, RestoreOptions *opts,
-		int numWorkers, bool append_data)
+				   int numWorkers, bool append_data)
 {
 	Archive		*AH;
 	bool		exit_code;
@@ -955,20 +956,20 @@ restoreAllDatabases(PGconn *conn, const char *dumpdirpath,
 
 	if (!conn)
 	{
-		pg_log_info("trying to connect postgres database to dump into out file");
+		pg_log_info("trying to connect database \"postgres\"  to dump into out file");
 
 		conn = connectDatabase("postgres", NULL, opts->cparams.pghost,
-				opts->cparams.pgport, opts->cparams.username, TRI_DEFAULT, false,
-				progname, NULL, NULL);
+							   opts->cparams.pgport, opts->cparams.username, TRI_DEFAULT,
+							   false, progname, NULL, NULL);
 
 		/* Try with template1. */
 		if (!conn)
 		{
-			pg_log_info("trying to connect template1 database as failed to connect to postgres to dump into out file");
+			pg_log_info("trying to connect database \"template1\" as failed to connect to database \"postgres\" to dump into out file");
 
 			conn = connectDatabase("template1", NULL, opts->cparams.pghost,
-					opts->cparams.pgport, opts->cparams.username, TRI_DEFAULT, false,
-					progname, NULL, NULL);
+								   opts->cparams.pgport, opts->cparams.username, TRI_DEFAULT,
+								   false, progname, NULL, NULL);
 
 			if (!conn)
 				pg_log_info("there is no database connection so consider pattern as simple name for exclude-database");
-- 
2.34.1