From df8b8968d4095f44acd6de03b4add65f9709b79d Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Mon, 12 Dec 2022 14:33:41 +0100 Subject: Order getopt arguments Order the letters in the arguments of getopt() and getopt_long(), as well as in the subsequent switch statements. In most cases, I used alphabetical with lower case first. In a few cases, existing different orders (e.g., upper case first) was kept to reduce the diff size. Discussion: https://www.postgresql.org/message-id/flat/3efd0fe8-351b-f836-9122-886002602357%40enterprisedb.com --- src/bin/scripts/clusterdb.c | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-) (limited to 'src/bin/scripts/clusterdb.c') diff --git a/src/bin/scripts/clusterdb.c b/src/bin/scripts/clusterdb.c index df1766679b5..0bda9415803 100644 --- a/src/bin/scripts/clusterdb.c +++ b/src/bin/scripts/clusterdb.c @@ -68,43 +68,43 @@ main(int argc, char *argv[]) handle_help_version_opts(argc, argv, "clusterdb", help); - while ((c = getopt_long(argc, argv, "h:p:U:wWeqd:at:v", long_options, &optindex)) != -1) + while ((c = getopt_long(argc, argv, "ad:eh:p:qt:U:vwW", long_options, &optindex)) != -1) { switch (c) { + case 'a': + alldb = true; + break; + case 'd': + dbname = pg_strdup(optarg); + break; + case 'e': + echo = true; + break; case 'h': host = pg_strdup(optarg); break; case 'p': port = pg_strdup(optarg); break; - case 'U': - username = pg_strdup(optarg); - break; - case 'w': - prompt_password = TRI_NO; - break; - case 'W': - prompt_password = TRI_YES; - break; - case 'e': - echo = true; - break; case 'q': quiet = true; break; - case 'd': - dbname = pg_strdup(optarg); - break; - case 'a': - alldb = true; - break; case 't': simple_string_list_append(&tables, optarg); break; + case 'U': + username = pg_strdup(optarg); + break; case 'v': verbose = true; break; + case 'w': + prompt_password = TRI_NO; + break; + case 'W': + prompt_password = TRI_YES; + break; case 2: maintenance_db = pg_strdup(optarg); break; -- cgit v1.2.3