fixup! src: use uint32_t for process initialization flags enum

addaleax · addaleax · commit c259b3b3aecf · 2023-01-31T17:35:25.000+01:00
diff --git a/src/node.cc b/src/node.cc
@@ -431,6 +431,9 @@ void ResetSignalHandlers() {
 #endif  // __POSIX__
 }
 
+// We use uint32_t since that can be accessed as a lock-free atomic
+// variable on all platforms that we support, which we require in
+// order for its value to be usable inside signal handlers.
 static std::atomic<uint32_t> init_process_flags = 0;
 static_assert(
     std::is_same_v<std::underlying_type_t<ProcessInitializationFlags::Flags>,

Original file line number	Diff line number	Diff line change
`@@ -431,6 +431,9 @@ void ResetSignalHandlers() {`
`431`	`431`	`#endif // __POSIX__`
`432`	`432`	`}`
`433`	`433`
	`434`	`+// We use uint32_t since that can be accessed as a lock-free atomic`
	`435`	`+// variable on all platforms that we support, which we require in`
	`436`	`+// order for its value to be usable inside signal handlers.`
`434`	`437`	`static std::atomic<uint32_t> init_process_flags = 0;`
`435`	`438`	`static_assert(`
`436`	`439`	`std::is_same_v<std::underlying_type_t<ProcessInitializationFlags::Flags>,`