Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 8d2c191

Browse files
committed
Remove unneeded volatile qualifiers from postmaster.c.
Several flags were marked volatile and in some cases used sig_atomic_t because they were accessed from signal handlers. After commit 7389aad, we can just use unqualified bool. Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us> Reviewed-by: Andres Freund <andres@anarazel.de> Discussion: https://postgr.es/m/CA%2BhUKGLMoeZNZY6gYdLUQmuoW_a8bKyLvtuZkd_zHcGVOfDzBA%40mail.gmail.com
1 parent e4e89eb commit 8d2c191

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/backend/postmaster/postmaster.c

+5-5
Original file line numberDiff line numberDiff line change
@@ -359,17 +359,17 @@ bool ClientAuthInProgress = false; /* T during new-client
359359
bool redirection_done = false; /* stderr redirected for syslogger? */
360360

361361
/* received START_AUTOVAC_LAUNCHER signal */
362-
static volatile sig_atomic_t start_autovac_launcher = false;
362+
static bool start_autovac_launcher = false;
363363

364364
/* the launcher needs to be signaled to communicate some condition */
365-
static volatile bool avlauncher_needs_signal = false;
365+
static bool avlauncher_needs_signal = false;
366366

367367
/* received START_WALRECEIVER signal */
368-
static volatile sig_atomic_t WalReceiverRequested = false;
368+
static bool WalReceiverRequested = false;
369369

370370
/* set when there's a worker that needs to be started up */
371-
static volatile bool StartWorkerNeeded = true;
372-
static volatile bool HaveCrashedWorker = false;
371+
static bool StartWorkerNeeded = true;
372+
static bool HaveCrashedWorker = false;
373373

374374
/* set when signals arrive */
375375
static volatile sig_atomic_t pending_pm_pmsignal;

0 commit comments

Comments
 (0)