Commit78fdb1e

committed

Mark ParallelMessagePending as sig_atomic_t

ParallelMessagePending was previously marked as a boolean which shouldbe fine on modern platforms, but the C standard recommends the use ofsig_atomic_t for variables manipulated in signal handlers.Author: Hayato KurodaDiscussion:https://postgr.es/m/TYAPR01MB58667C15A95A234720F4F876F5529@TYAPR01MB5866.jpnprd01.prod.outlook.com

1 parente1e6f8f commit78fdb1eCopy full SHA for 78fdb1e

File tree

-2

lines changed

-2

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -113,7 +113,7 @@ typedef struct FixedParallelState`
`113`	`113`	`intParallelWorkerNumber=-1;`
`114`	`114`
`115`	`115`	`/* Is there a parallel message pending which we need to receive? */`
`116`		`-volatileboolParallelMessagePending= false;`
	`116`	`+volatilesig_atomic_tParallelMessagePending= false;`
`117`	`117`
`118`	`118`	`/* Are we initializing a parallel worker? */`
`119`	`119`	`boolInitializingParallelWorker= false;`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ typedef struct ParallelWorkerContext`
`54`	`54`	`shm_toc*toc;`
`55`	`55`	`}ParallelWorkerContext;`
`56`	`56`
`57`		`-externPGDLLIMPORTvolatileboolParallelMessagePending;`
	`57`	`+externPGDLLIMPORTvolatilesig_atomic_tParallelMessagePending;`
`58`	`58`	`externPGDLLIMPORTintParallelWorkerNumber;`
`59`	`59`	`externPGDLLIMPORTboolInitializingParallelWorker;`
`60`	`60`

Comments

(0)