From 5713678c7ada828b333ae5dc017e4a0c15c04d1c Mon Sep 17 00:00:00 2001
From: pluris <zighart8456@naver.com>
Date: Sun, 17 Sep 2023 02:09:25 +0900
Subject: [PATCH] src: move const variable in `node_file.h` to `node_file.cc`

---
 src/node_file.cc | 20 ++++++++++++++------
 src/node_file.h  |  8 --------
 2 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/node_file.cc b/src/node_file.cc
index ee97188d266ed2..7ad76f53f47446 100644
--- a/src/node_file.cc
+++ b/src/node_file.cc
@@ -2909,11 +2909,18 @@ BindingData::FilePathIsFileReturnType BindingData::FilePathIsFile(
   return BindingData::FilePathIsFileReturnType::kIsNotFile;
 }
 
+namespace {
+// define the final index of the algorithm resolution
+// when packageConfig.main is defined.
+constexpr uint8_t legacy_main_extensions_with_main_end = 7;
+// define the final index of the algorithm resolution
+// when packageConfig.main is NOT defined
+constexpr uint8_t legacy_main_extensions_package_fallback_end = 10;
 // the possible file extensions that should be tested
 // 0-6: when packageConfig.main is defined
 // 7-9: when packageConfig.main is NOT defined,
 //      or when the previous case didn't found the file
-const std::array<std::string, 10> BindingData::legacy_main_extensions = {
+constexpr char* legacy_main_extensions[] = {
     "",
     ".js",
     ".json",
@@ -2924,6 +2931,7 @@ const std::array<std::string, 10> BindingData::legacy_main_extensions = {
     ".js",
     ".json",
     ".node"};
+}
 
 void BindingData::LegacyMainResolve(const FunctionCallbackInfo<Value>& args) {
   CHECK_GE(args.Length(), 1);
@@ -2965,9 +2973,9 @@ void BindingData::LegacyMainResolve(const FunctionCallbackInfo<Value>& args) {
 
     FromNamespacedPath(&initial_file_path);
 
-    for (int i = 0; i < BindingData::legacy_main_extensions_with_main_end;
+    for (int i = 0; i < legacy_main_extensions_with_main_end;
          i++) {
-      file_path = initial_file_path + BindingData::legacy_main_extensions[i];
+      file_path = initial_file_path + legacy_main_extensions[i];
 
       switch (FilePathIsFile(env, file_path)) {
         case BindingData::FilePathIsFileReturnType::kIsFile:
@@ -3000,10 +3008,10 @@ void BindingData::LegacyMainResolve(const FunctionCallbackInfo<Value>& args) {
 
   FromNamespacedPath(&initial_file_path);
 
-  for (int i = BindingData::legacy_main_extensions_with_main_end;
-       i < BindingData::legacy_main_extensions_package_fallback_end;
+  for (int i = legacy_main_extensions_with_main_end;
+       i < legacy_main_extensions_package_fallback_end;
        i++) {
-    file_path = initial_file_path + BindingData::legacy_main_extensions[i];
+    file_path = initial_file_path + legacy_main_extensions[i];
 
     switch (FilePathIsFile(env, file_path)) {
       case BindingData::FilePathIsFileReturnType::kIsFile:
diff --git a/src/node_file.h b/src/node_file.h
index 0fa2da0b4f7f8a..f608da5b7a17e2 100644
--- a/src/node_file.h
+++ b/src/node_file.h
@@ -102,14 +102,6 @@ class BindingData : public SnapshotableObject {
 
   static FilePathIsFileReturnType FilePathIsFile(Environment* env,
                                                  const std::string& file_path);
-
-  static const std::array<std::string, 10> legacy_main_extensions;
-  // define the final index of the algorithm resolution
-  // when packageConfig.main is defined.
-  static const uint8_t legacy_main_extensions_with_main_end = 7;
-  // define the final index of the algorithm resolution
-  // when packageConfig.main is NOT defined
-  static const uint8_t legacy_main_extensions_package_fallback_end = 10;
 };
 
 // structure used to store state during a complex operation, e.g., mkdirp.