src: replace kPathSeparator with std::filesystem

Author: anonrig

src/compile_cache.cc

@@ -205,8 +205,9 @@ CompileCacheEntry* CompileCacheHandler::GetOrInsert(
   result->code_hash = code_hash;
   result->code_size = code_utf8.length();
   result->cache_key = key;
-  result->cache_filename =
-      compile_cache_dir_ + kPathSeparator + Uint32ToHex(result->cache_key);
+  result->cache_filename = compile_cache_dir_ +
+                           std::filesystem::path::preferred_separator +
+                           Uint32ToHex(result->cache_key);
   result->source_filename = filename_utf8.ToString();
   result->cache = nullptr;
   result->type = type;

src/compile_cache.h

@@ -4,6 +4,7 @@
 #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
 
 #include <cinttypes>
+#include <filesystem>
 #include <memory>
 #include <string>
 #include <unordered_map>

src/env.cc

@@ -713,7 +713,8 @@ std::string Environment::GetCwd(const std::string& exec_path) {
 
   // This can fail if the cwd is deleted. In that case, fall back to
   // exec_path.
-  return exec_path.substr(0, exec_path.find_last_of(kPathSeparator));
+  return exec_path.substr(
+      0, exec_path.find_last_of(std::filesystem::path::preferred_separator));
 }
 
 void Environment::add_refs(int64_t diff) {
@@ -2064,7 +2065,8 @@ size_t Environment::NearHeapLimitCallback(void* data,
     dir = Environment::GetCwd(env->exec_path_);
   }
   DiagnosticFilename name(env, "Heap", "heapsnapshot");
-  std::string filename = dir + kPathSeparator + (*name);
+  std::string filename =
+      dir + std::filesystem::path::preferred_separator + (*name);
 
   Debug(env, DebugCategory::DIAGNOSTICS, "Start generating %s...\n", *name);
 

src/env.h

@@ -57,6 +57,7 @@
 #include <array>
 #include <atomic>
 #include <cstdint>
+#include <filesystem>
 #include <functional>
 #include <list>
 #include <memory>

src/inspector_profiler.cc

@@ -248,7 +248,7 @@ void V8ProfilerConnection::WriteProfile(simdjson::ondemand::object* result) {
 
   std::string filename = GetFilename();
   DCHECK(!filename.empty());
-  std::string path = directory + kPathSeparator + filename;
+  auto path = std::filesystem::path(directory) / filename;
 
   WriteResult(env_, path.c_str(), profile);
 }
@@ -304,7 +304,7 @@ void V8CoverageConnection::WriteProfile(simdjson::ondemand::object* result) {
 
   std::string filename = GetFilename();
   DCHECK(!filename.empty());
-  std::string path = directory + kPathSeparator + filename;
+  auto path = std::filesystem::path(directory) / filename;
 
   // Only insert source map cache when there's source map data at all.
   if (!source_map_cache_v->IsUndefined()) {

src/inspector_profiler.h

@@ -7,6 +7,7 @@
 #error("This header can only be used when inspector is enabled")
 #endif
 
+#include <filesystem>
 #include <optional>
 #include <unordered_set>
 #include "inspector_agent.h"

src/node_file.cc

@@ -76,15 +76,10 @@ using v8::Value;
 # define S_ISDIR(mode)  (((mode) & S_IFMT) == S_IFDIR)
 #endif
 
-#ifdef __POSIX__
-constexpr char kPathSeparator = '/';
-#else
-const char* const kPathSeparator = "\\/";
-#endif
-
 std::string Basename(const std::string& str, const std::string& extension) {
   // Remove everything leading up to and including the final path separator.
-  std::string::size_type pos = str.find_last_of(kPathSeparator);
+  std::string::size_type pos =
+      str.find_last_of(std::filesystem::path::preferred_separator);
 
   // Starting index for the resulting string
   std::size_t start_pos = 0;
@@ -1614,8 +1609,10 @@ int MKDirpSync(uv_loop_t* loop,
           return err;
         }
         case UV_ENOENT: {
-          std::string dirname = next_path.substr(0,
-                                        next_path.find_last_of(kPathSeparator));
+          std::string dirname =
+              next_path.substr(0,
+                               next_path.find_last_of(
+                                   std::filesystem::path::preferred_separator));
           if (dirname != next_path) {
             req_wrap->continuation_data()->PushPath(std::move(next_path));
             req_wrap->continuation_data()->PushPath(std::move(dirname));
@@ -1694,8 +1691,8 @@ int MKDirpAsync(uv_loop_t* loop,
           break;
         }
         case UV_ENOENT: {
-          std::string dirname = path.substr(0,
-                                            path.find_last_of(kPathSeparator));
+          std::string dirname = path.substr(
+              0, path.find_last_of(std::filesystem::path::preferred_separator));
           if (dirname != path) {
             req_wrap->continuation_data()->PushPath(path);
             req_wrap->continuation_data()->PushPath(std::move(dirname));

src/node_file.h

@@ -3,6 +3,7 @@
 
 #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
 
+#include <filesystem>
 #include <optional>
 #include "aliased_buffer.h"
 #include "node_messaging.h"

src/node_report.cc

@@ -887,9 +887,9 @@ std::string TriggerNodeReport(Isolate* isolate,
       report_directory = per_process::cli_options->report_directory;
     }
     // Regular file. Append filename to directory path if one was specified
-    if (report_directory.length() > 0) {
+    if (!report_directory.empty()) {
       std::string pathname = report_directory;
-      pathname += kPathSeparator;
+      pathname += std::filesystem::path::preferred_separator;
       pathname += filename;
       outfile.open(pathname, std::ios::out | std::ios::binary);
     } else {

src/node_report.h

@@ -13,6 +13,7 @@
 #include <unistd.h>
 #endif
 
+#include <filesystem>
 #include <iomanip>
 #include <sstream>