From d43a10b5ba46b010c8e075b1062f9f30eb013498 Mon Sep 17 00:00:00 2001 From: Andres Freund Date: Wed, 17 Apr 2024 14:27:03 -0700 Subject: [PATCH v1 1/3] static const: Convert struct option arrays --- src/bin/initdb/initdb.c | 2 +- src/bin/pg_amcheck/pg_amcheck.c | 2 +- src/bin/pg_archivecleanup/pg_archivecleanup.c | 2 +- src/bin/pg_basebackup/pg_basebackup.c | 2 +- src/bin/pg_basebackup/pg_createsubscriber.c | 2 +- src/bin/pg_basebackup/pg_receivewal.c | 2 +- src/bin/pg_basebackup/pg_recvlogical.c | 2 +- src/bin/pg_checksums/pg_checksums.c | 2 +- src/bin/pg_combinebackup/pg_combinebackup.c | 2 +- src/bin/pg_controldata/pg_controldata.c | 2 +- src/bin/pg_ctl/pg_ctl.c | 2 +- src/bin/pg_dump/pg_dump.c | 2 +- src/bin/pg_dump/pg_dumpall.c | 2 +- src/bin/pg_resetwal/pg_resetwal.c | 2 +- src/bin/pg_rewind/pg_rewind.c | 2 +- src/bin/pg_test_fsync/pg_test_fsync.c | 2 +- src/bin/pg_test_timing/pg_test_timing.c | 2 +- src/bin/pg_upgrade/option.c | 2 +- src/bin/pg_verifybackup/pg_verifybackup.c | 2 +- src/bin/pg_waldump/pg_waldump.c | 2 +- src/bin/pg_walsummary/pg_walsummary.c | 2 +- src/bin/pgbench/pgbench.c | 2 +- src/bin/psql/startup.c | 2 +- src/bin/scripts/clusterdb.c | 2 +- src/bin/scripts/createdb.c | 2 +- src/bin/scripts/createuser.c | 2 +- src/bin/scripts/dropdb.c | 2 +- src/bin/scripts/dropuser.c | 2 +- src/bin/scripts/pg_isready.c | 2 +- src/bin/scripts/reindexdb.c | 2 +- src/bin/scripts/vacuumdb.c | 2 +- contrib/btree_gist/btree_interval.c | 2 +- contrib/oid2name/oid2name.c | 2 +- contrib/vacuumlo/vacuumlo.c | 2 +- src/interfaces/ecpg/preproc/ecpg.c | 2 +- src/test/regress/pg_regress.c | 2 +- 36 files changed, 36 insertions(+), 36 deletions(-) diff --git a/src/bin/initdb/initdb.c b/src/bin/initdb/initdb.c index 30e17bd1d1e..5d4044d5a90 100644 --- a/src/bin/initdb/initdb.c +++ b/src/bin/initdb/initdb.c @@ -3093,7 +3093,7 @@ initialize_data_directory(void) int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"pgdata", required_argument, NULL, 'D'}, {"encoding", required_argument, NULL, 'E'}, {"locale", required_argument, NULL, 1}, diff --git a/src/bin/pg_amcheck/pg_amcheck.c b/src/bin/pg_amcheck/pg_amcheck.c index 7e3101704d4..bb100022a0d 100644 --- a/src/bin/pg_amcheck/pg_amcheck.c +++ b/src/bin/pg_amcheck/pg_amcheck.c @@ -233,7 +233,7 @@ main(int argc, char *argv[]) uint64 relprogress = 0; int pattern_id; - static struct option long_options[] = { + static const struct option long_options[] = { /* Connection options */ {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, diff --git a/src/bin/pg_archivecleanup/pg_archivecleanup.c b/src/bin/pg_archivecleanup/pg_archivecleanup.c index 07bf356b70c..6cdf471597e 100644 --- a/src/bin/pg_archivecleanup/pg_archivecleanup.c +++ b/src/bin/pg_archivecleanup/pg_archivecleanup.c @@ -285,7 +285,7 @@ usage(void) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"clean-backup-history", no_argument, NULL, 'b'}, {"debug", no_argument, NULL, 'd'}, {"dry-run", no_argument, NULL, 'n'}, diff --git a/src/bin/pg_basebackup/pg_basebackup.c b/src/bin/pg_basebackup/pg_basebackup.c index 8f3dd04fd22..0220614993f 100644 --- a/src/bin/pg_basebackup/pg_basebackup.c +++ b/src/bin/pg_basebackup/pg_basebackup.c @@ -2354,7 +2354,7 @@ BaseBackup(char *compression_algorithm, char *compression_detail, int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"help", no_argument, NULL, '?'}, {"version", no_argument, NULL, 'V'}, {"pgdata", required_argument, NULL, 'D'}, diff --git a/src/bin/pg_basebackup/pg_createsubscriber.c b/src/bin/pg_basebackup/pg_createsubscriber.c index 90cc580811d..cbb4b92be87 100644 --- a/src/bin/pg_basebackup/pg_createsubscriber.c +++ b/src/bin/pg_basebackup/pg_createsubscriber.c @@ -1754,7 +1754,7 @@ enable_subscription(PGconn *conn, const struct LogicalRepInfo *dbinfo) int main(int argc, char **argv) { - static struct option long_options[] = + static const struct option long_options[] = { {"database", required_argument, NULL, 'd'}, {"pgdata", required_argument, NULL, 'D'}, diff --git a/src/bin/pg_basebackup/pg_receivewal.c b/src/bin/pg_basebackup/pg_receivewal.c index 555f0175f0e..857460e7977 100644 --- a/src/bin/pg_basebackup/pg_receivewal.c +++ b/src/bin/pg_basebackup/pg_receivewal.c @@ -623,7 +623,7 @@ sigexit_handler(SIGNAL_ARGS) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"help", no_argument, NULL, '?'}, {"version", no_argument, NULL, 'V'}, {"directory", required_argument, NULL, 'D'}, diff --git a/src/bin/pg_basebackup/pg_recvlogical.c b/src/bin/pg_basebackup/pg_recvlogical.c index 3db520ed38b..fb15a09f72d 100644 --- a/src/bin/pg_basebackup/pg_recvlogical.c +++ b/src/bin/pg_basebackup/pg_recvlogical.c @@ -691,7 +691,7 @@ sighup_handler(SIGNAL_ARGS) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { /* general options */ {"file", required_argument, NULL, 'f'}, {"fsync-interval", required_argument, NULL, 'F'}, diff --git a/src/bin/pg_checksums/pg_checksums.c b/src/bin/pg_checksums/pg_checksums.c index 9e6fd435f60..4f49b3a74a6 100644 --- a/src/bin/pg_checksums/pg_checksums.c +++ b/src/bin/pg_checksums/pg_checksums.c @@ -432,7 +432,7 @@ scan_directory(const char *basedir, const char *subdir, bool sizeonly) int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"check", no_argument, NULL, 'c'}, {"pgdata", required_argument, NULL, 'D'}, {"disable", no_argument, NULL, 'd'}, diff --git a/src/bin/pg_combinebackup/pg_combinebackup.c b/src/bin/pg_combinebackup/pg_combinebackup.c index 2788c78fdd4..080c9244e8a 100644 --- a/src/bin/pg_combinebackup/pg_combinebackup.c +++ b/src/bin/pg_combinebackup/pg_combinebackup.c @@ -121,7 +121,7 @@ static void slurp_file(int fd, char *filename, StringInfo buf, int maxlen); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"debug", no_argument, NULL, 'd'}, {"dry-run", no_argument, NULL, 'n'}, {"no-sync", no_argument, NULL, 'N'}, diff --git a/src/bin/pg_controldata/pg_controldata.c b/src/bin/pg_controldata/pg_controldata.c index 93e0837947c..b4117e5d990 100644 --- a/src/bin/pg_controldata/pg_controldata.c +++ b/src/bin/pg_controldata/pg_controldata.c @@ -88,7 +88,7 @@ wal_level_str(WalLevel wal_level) int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"pgdata", required_argument, NULL, 'D'}, {NULL, 0, NULL, 0} }; diff --git a/src/bin/pg_ctl/pg_ctl.c b/src/bin/pg_ctl/pg_ctl.c index 405e223c190..5d18781eaf4 100644 --- a/src/bin/pg_ctl/pg_ctl.c +++ b/src/bin/pg_ctl/pg_ctl.c @@ -2189,7 +2189,7 @@ get_control_dbstate(void) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"help", no_argument, NULL, '?'}, {"version", no_argument, NULL, 'V'}, {"log", required_argument, NULL, 'l'}, diff --git a/src/bin/pg_dump/pg_dump.c b/src/bin/pg_dump/pg_dump.c index c52e961b309..63f941ec344 100644 --- a/src/bin/pg_dump/pg_dump.c +++ b/src/bin/pg_dump/pg_dump.c @@ -374,7 +374,7 @@ main(int argc, char **argv) static DumpOptions dopt; - static struct option long_options[] = { + static const struct option long_options[] = { {"data-only", no_argument, NULL, 'a'}, {"blobs", no_argument, NULL, 'b'}, {"large-objects", no_argument, NULL, 'b'}, diff --git a/src/bin/pg_dump/pg_dumpall.c b/src/bin/pg_dump/pg_dumpall.c index 73337f33923..0bcc4079547 100644 --- a/src/bin/pg_dump/pg_dumpall.c +++ b/src/bin/pg_dump/pg_dumpall.c @@ -126,7 +126,7 @@ static SimpleStringList database_exclude_names = {NULL, NULL}; int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"data-only", no_argument, NULL, 'a'}, {"clean", no_argument, NULL, 'c'}, {"encoding", required_argument, NULL, 'E'}, diff --git a/src/bin/pg_resetwal/pg_resetwal.c b/src/bin/pg_resetwal/pg_resetwal.c index e9dcb5a6d89..184cab2a842 100644 --- a/src/bin/pg_resetwal/pg_resetwal.c +++ b/src/bin/pg_resetwal/pg_resetwal.c @@ -93,7 +93,7 @@ static void usage(void); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"commit-timestamp-ids", required_argument, NULL, 'c'}, {"pgdata", required_argument, NULL, 'D'}, {"epoch", required_argument, NULL, 'e'}, diff --git a/src/bin/pg_rewind/pg_rewind.c b/src/bin/pg_rewind/pg_rewind.c index 8449ae78ef7..675cb5e739c 100644 --- a/src/bin/pg_rewind/pg_rewind.c +++ b/src/bin/pg_rewind/pg_rewind.c @@ -120,7 +120,7 @@ usage(const char *progname) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"help", no_argument, NULL, '?'}, {"target-pgdata", required_argument, NULL, 'D'}, {"write-recovery-conf", no_argument, NULL, 'R'}, diff --git a/src/bin/pg_test_fsync/pg_test_fsync.c b/src/bin/pg_test_fsync/pg_test_fsync.c index 5c0da425fbb..071a3bc2599 100644 --- a/src/bin/pg_test_fsync/pg_test_fsync.c +++ b/src/bin/pg_test_fsync/pg_test_fsync.c @@ -147,7 +147,7 @@ main(int argc, char *argv[]) static void handle_args(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"filename", required_argument, NULL, 'f'}, {"secs-per-test", required_argument, NULL, 's'}, {NULL, 0, NULL, 0} diff --git a/src/bin/pg_test_timing/pg_test_timing.c b/src/bin/pg_test_timing/pg_test_timing.c index c29d6f87629..3996889d2fd 100644 --- a/src/bin/pg_test_timing/pg_test_timing.c +++ b/src/bin/pg_test_timing/pg_test_timing.c @@ -42,7 +42,7 @@ main(int argc, char *argv[]) static void handle_args(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"duration", required_argument, NULL, 'd'}, {NULL, 0, NULL, 0} }; diff --git a/src/bin/pg_upgrade/option.c b/src/bin/pg_upgrade/option.c index 548ea4e6236..c841ddea776 100644 --- a/src/bin/pg_upgrade/option.c +++ b/src/bin/pg_upgrade/option.c @@ -38,7 +38,7 @@ UserOpts user_opts; void parseCommandLine(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"old-datadir", required_argument, NULL, 'd'}, {"new-datadir", required_argument, NULL, 'D'}, {"old-bindir", required_argument, NULL, 'b'}, diff --git a/src/bin/pg_verifybackup/pg_verifybackup.c b/src/bin/pg_verifybackup/pg_verifybackup.c index fd610c20a65..b34c0bd6bde 100644 --- a/src/bin/pg_verifybackup/pg_verifybackup.c +++ b/src/bin/pg_verifybackup/pg_verifybackup.c @@ -176,7 +176,7 @@ static uint64 done_size = 0; int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"exit-on-error", no_argument, NULL, 'e'}, {"ignore", required_argument, NULL, 'i'}, {"manifest-path", required_argument, NULL, 'm'}, diff --git a/src/bin/pg_waldump/pg_waldump.c b/src/bin/pg_waldump/pg_waldump.c index 1f9403fc5cf..321299c0e00 100644 --- a/src/bin/pg_waldump/pg_waldump.c +++ b/src/bin/pg_waldump/pg_waldump.c @@ -802,7 +802,7 @@ main(int argc, char **argv) char *waldir = NULL; char *errormsg; - static struct option long_options[] = { + static const struct option long_options[] = { {"bkp-details", no_argument, NULL, 'b'}, {"block", required_argument, NULL, 'B'}, {"end", required_argument, NULL, 'e'}, diff --git a/src/bin/pg_walsummary/pg_walsummary.c b/src/bin/pg_walsummary/pg_walsummary.c index daf6cd14ce7..9a5785bbefc 100644 --- a/src/bin/pg_walsummary/pg_walsummary.c +++ b/src/bin/pg_walsummary/pg_walsummary.c @@ -52,7 +52,7 @@ static void walsummary_error_callback(void *callback_arg, char *fmt,...) pg_attr int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"individual", no_argument, NULL, 'i'}, {"quiet", no_argument, NULL, 'q'}, {NULL, 0, NULL, 0} diff --git a/src/bin/pgbench/pgbench.c b/src/bin/pgbench/pgbench.c index af776b31d8f..1e99f6c31e1 100644 --- a/src/bin/pgbench/pgbench.c +++ b/src/bin/pgbench/pgbench.c @@ -6616,7 +6616,7 @@ set_random_seed(const char *seed) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { /* systematic long/short named options */ {"builtin", required_argument, NULL, 'b'}, {"client", required_argument, NULL, 'c'}, diff --git a/src/bin/psql/startup.c b/src/bin/psql/startup.c index 036caaec2ff..b22a1cf8c52 100644 --- a/src/bin/psql/startup.c +++ b/src/bin/psql/startup.c @@ -482,7 +482,7 @@ error: static void parse_psql_options(int argc, char *argv[], struct adhoc_opts *options) { - static struct option long_options[] = + static const struct option long_options[] = { {"echo-all", no_argument, NULL, 'a'}, {"no-align", no_argument, NULL, 'A'}, diff --git a/src/bin/scripts/clusterdb.c b/src/bin/scripts/clusterdb.c index 3503a3bb584..90810946c62 100644 --- a/src/bin/scripts/clusterdb.c +++ b/src/bin/scripts/clusterdb.c @@ -30,7 +30,7 @@ static void help(const char *progname); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, {"username", required_argument, NULL, 'U'}, diff --git a/src/bin/scripts/createdb.c b/src/bin/scripts/createdb.c index 007061e756f..55f1be58a9e 100644 --- a/src/bin/scripts/createdb.c +++ b/src/bin/scripts/createdb.c @@ -23,7 +23,7 @@ static void help(const char *progname); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, {"username", required_argument, NULL, 'U'}, diff --git a/src/bin/scripts/createuser.c b/src/bin/scripts/createuser.c index f5f03c1145f..4b880815fba 100644 --- a/src/bin/scripts/createuser.c +++ b/src/bin/scripts/createuser.c @@ -27,7 +27,7 @@ static void help(const char *progname); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"with-admin", required_argument, NULL, 'a'}, {"connection-limit", required_argument, NULL, 'c'}, {"createdb", no_argument, NULL, 'd'}, diff --git a/src/bin/scripts/dropdb.c b/src/bin/scripts/dropdb.c index 57656ba8988..bf896a2919b 100644 --- a/src/bin/scripts/dropdb.c +++ b/src/bin/scripts/dropdb.c @@ -25,7 +25,7 @@ main(int argc, char *argv[]) { static int if_exists = 0; - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, {"username", required_argument, NULL, 'U'}, diff --git a/src/bin/scripts/dropuser.c b/src/bin/scripts/dropuser.c index 233ccd288b7..723358b5470 100644 --- a/src/bin/scripts/dropuser.c +++ b/src/bin/scripts/dropuser.c @@ -26,7 +26,7 @@ main(int argc, char *argv[]) { static int if_exists = 0; - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, {"username", required_argument, NULL, 'U'}, diff --git a/src/bin/scripts/pg_isready.c b/src/bin/scripts/pg_isready.c index 100589da147..f56532678be 100644 --- a/src/bin/scripts/pg_isready.c +++ b/src/bin/scripts/pg_isready.c @@ -55,7 +55,7 @@ main(int argc, char **argv) * connecting with invalid params */ - static struct option long_options[] = { + static const struct option long_options[] = { {"dbname", required_argument, NULL, 'd'}, {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, diff --git a/src/bin/scripts/reindexdb.c b/src/bin/scripts/reindexdb.c index 904c196fbba..85ba625b73d 100644 --- a/src/bin/scripts/reindexdb.c +++ b/src/bin/scripts/reindexdb.c @@ -60,7 +60,7 @@ static void help(const char *progname); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, {"username", required_argument, NULL, 'U'}, diff --git a/src/bin/scripts/vacuumdb.c b/src/bin/scripts/vacuumdb.c index 7138c6e97e4..169a5960ae0 100644 --- a/src/bin/scripts/vacuumdb.c +++ b/src/bin/scripts/vacuumdb.c @@ -96,7 +96,7 @@ static char *escape_quotes(const char *src); int main(int argc, char *argv[]) { - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"port", required_argument, NULL, 'p'}, {"username", required_argument, NULL, 'U'}, diff --git a/contrib/btree_gist/btree_interval.c b/contrib/btree_gist/btree_interval.c index b0afdf02bb5..156f2cebac5 100644 --- a/contrib/btree_gist/btree_interval.c +++ b/contrib/btree_gist/btree_interval.c @@ -113,7 +113,7 @@ static const gbtree_ninfo tinfo = Interval * abs_interval(Interval *a) { - static Interval zero = {0, 0, 0}; + static const Interval zero = {0, 0, 0}; if (DatumGetBool(DirectFunctionCall2(interval_lt, IntervalPGetDatum(a), diff --git a/contrib/oid2name/oid2name.c b/contrib/oid2name/oid2name.c index e8c1e2c97bd..c2785848f55 100644 --- a/contrib/oid2name/oid2name.c +++ b/contrib/oid2name/oid2name.c @@ -62,7 +62,7 @@ void sql_exec_dumpalltbspc(PGconn *conn, struct options *opts); void get_opts(int argc, char **argv, struct options *my_opts) { - static struct option long_options[] = { + static const struct option long_options[] = { {"dbname", required_argument, NULL, 'd'}, {"host", required_argument, NULL, 'h'}, {"host", required_argument, NULL, 'H'}, /* deprecated */ diff --git a/contrib/vacuumlo/vacuumlo.c b/contrib/vacuumlo/vacuumlo.c index 0d99428dec6..b5ba5acd67d 100644 --- a/contrib/vacuumlo/vacuumlo.c +++ b/contrib/vacuumlo/vacuumlo.c @@ -436,7 +436,7 @@ usage(const char *progname) int main(int argc, char **argv) { - static struct option long_options[] = { + static const struct option long_options[] = { {"host", required_argument, NULL, 'h'}, {"limit", required_argument, NULL, 'l'}, {"dry-run", no_argument, NULL, 'n'}, diff --git a/src/interfaces/ecpg/preproc/ecpg.c b/src/interfaces/ecpg/preproc/ecpg.c index 73c37631acc..3f744e7995d 100644 --- a/src/interfaces/ecpg/preproc/ecpg.c +++ b/src/interfaces/ecpg/preproc/ecpg.c @@ -128,7 +128,7 @@ add_preprocessor_define(char *define) int main(int argc, char *const argv[]) { - static struct option ecpg_options[] = { + static const struct option ecpg_options[] = { {"regression", no_argument, NULL, ECPG_GETOPT_LONG_REGRESSION}, {NULL, 0, NULL, 0} }; diff --git a/src/test/regress/pg_regress.c b/src/test/regress/pg_regress.c index 06f6775fc65..f0693463ee0 100644 --- a/src/test/regress/pg_regress.c +++ b/src/test/regress/pg_regress.c @@ -2066,7 +2066,7 @@ regression_main(int argc, char *argv[], test_start_function startfunc, postprocess_result_function postfunc) { - static struct option long_options[] = { + static const struct option long_options[] = { {"help", no_argument, NULL, 'h'}, {"version", no_argument, NULL, 'V'}, {"dbname", required_argument, NULL, 1}, -- 2.44.0.279.g3bd955d269