src: mark some global state as const

Mark some global variables as `const` or `constexpr`.

PR-URL: #25052
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: James M Snell <[email protected]>

Author: addaleax

src/env.cc

@@ -65,7 +65,7 @@ class TraceEventScope {
 };
 
 int const Environment::kNodeContextTag = 0x6e6f64;
-void* Environment::kNodeContextTagPtr = const_cast<void*>(
+void* const Environment::kNodeContextTagPtr = const_cast<void*>(
     static_cast<const void*>(&Environment::kNodeContextTag));
 
 IsolateData::IsolateData(Isolate* isolate,

src/env.h

@@ -971,7 +971,7 @@ class Environment {
   uint64_t thread_id_ = 0;
   std::unordered_set<worker::Worker*> sub_worker_contexts_;
 
-  static void* kNodeContextTagPtr;
+  static void* const kNodeContextTagPtr;
   static int const kNodeContextTag;
 
 #if HAVE_INSPECTOR

src/node_file.cc

@@ -81,9 +81,9 @@ using v8::Value;
 #endif
 
 #ifdef __POSIX__
-const char* kPathSeparator = "/";
+constexpr char kPathSeparator = '/';
 #else
-const char* kPathSeparator = "\\/";
+const char* const kPathSeparator = "\\/";
 #endif
 
 #define GET_OFFSET(a) ((a)->IsNumber() ? (a).As<Integer>()->Value() : -1)

src/node_http_parser_llhttp.cc

@@ -4,7 +4,7 @@
 
 namespace node {
 
-const char* llhttp_version =
+const char* const llhttp_version =
     NODE_STRINGIFY(LLHTTP_VERSION_MAJOR)
     "."
     NODE_STRINGIFY(LLHTTP_VERSION_MINOR)

src/node_http_parser_traditional.cc

@@ -6,7 +6,7 @@
 
 namespace node {
 
-const char* http_parser_version =
+const char* const http_parser_version =
   NODE_STRINGIFY(HTTP_PARSER_VERSION_MAJOR)
   "."
   NODE_STRINGIFY(HTTP_PARSER_VERSION_MINOR)

src/node_internals.h

@@ -697,8 +697,8 @@ static inline const char* errno_string(int errorno) {
 
 extern double prog_start_time;
 
-extern const char* llhttp_version;
-extern const char* http_parser_version;
+extern const char* const llhttp_version;
+extern const char* const http_parser_version;
 
 void Abort(const v8::FunctionCallbackInfo<v8::Value>& args);
 void Chdir(const v8::FunctionCallbackInfo<v8::Value>& args);

src/node_postmortem_metadata.cc

@@ -60,6 +60,6 @@ int GenDebugSymbols() {
   return 1;
 }
 
-int debug_symbols_generated = GenDebugSymbols();
+const int debug_symbols_generated = GenDebugSymbols();
 
 }  // namespace node