From 0b1b377569692ef85f46411854639ffd936505d6 Mon Sep 17 00:00:00 2001 From: Bruce Momjian Date: Fri, 27 Sep 2002 03:59:00 +0000 Subject: [PATCH] Have -d5 -d0 not affect the extra variables set with -d5. --- src/backend/tcop/postgres.c | 39 ++++++++++++++++++++----------------- 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c index 36022bcb32..acda7a9260 100644 --- a/src/backend/tcop/postgres.c +++ b/src/backend/tcop/postgres.c @@ -8,7 +8,7 @@ * * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.296 2002/09/27 03:34:15 momjian Exp $ + * $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.297 2002/09/27 03:59:00 momjian Exp $ * * NOTES * this is the "main" module of the postgres backend and @@ -1136,6 +1136,7 @@ PostgresMain(int argc, char *argv[], const char *username) const char *DBName = NULL; bool secure; int errs = 0; + int debug_flag = 0; GucContext ctx; GucSource gucsource; char *tmp; @@ -1250,6 +1251,7 @@ PostgresMain(int argc, char *argv[], const char *username) case 'd': /* debug level */ { + debug_flag = atoi(optarg); /* Set server debugging level. */ if (atoi(optarg) != 0) { @@ -1259,25 +1261,10 @@ PostgresMain(int argc, char *argv[], const char *username) SetConfigOption("server_min_messages", debugstr, ctx, gucsource); pfree(debugstr); - /* - * -d is not the same as setting - * client_min_messages because it enables other - * output options. - */ - if (atoi(optarg) >= 1) - SetConfigOption("log_connections", "true", ctx, gucsource); - if (atoi(optarg) >= 2) - SetConfigOption("log_statement", "true", ctx, gucsource); - if (atoi(optarg) >= 3) - SetConfigOption("debug_print_parse", "true", ctx, gucsource); - if (atoi(optarg) >= 4) - SetConfigOption("debug_print_plan", "true", ctx, gucsource); - if (atoi(optarg) >= 5) - SetConfigOption("debug_print_rewritten", "true", ctx, gucsource); } else /* - * -d 0 allows user to prevent postmaster debug + * -d0 allows user to prevent postmaster debug * from propagating to backend. It would be nice * to set it to the postgresql.conf value here. */ @@ -1520,6 +1507,22 @@ PostgresMain(int argc, char *argv[], const char *username) break; } + /* + * -d is not the same as setting + * server_min_messages because it enables other + * output options. + */ + if (debug_flag >= 1) + SetConfigOption("log_connections", "true", ctx, gucsource); + if (debug_flag >= 2) + SetConfigOption("log_statement", "true", ctx, gucsource); + if (debug_flag >= 3) + SetConfigOption("debug_print_parse", "true", ctx, gucsource); + if (debug_flag >= 4) + SetConfigOption("debug_print_plan", "true", ctx, gucsource); + if (debug_flag >= 5) + SetConfigOption("debug_print_rewritten", "true", ctx, gucsource); + /* * Post-processing for command line options. */ @@ -1698,7 +1701,7 @@ PostgresMain(int argc, char *argv[], const char *username) if (!IsUnderPostmaster) { puts("\nPOSTGRES backend interactive interface "); - puts("$Revision: 1.296 $ $Date: 2002/09/27 03:34:15 $\n"); + puts("$Revision: 1.297 $ $Date: 2002/09/27 03:59:00 $\n"); } /* -- 2.40.0