[d8/modules] Factor out path and string handling logic

src/d8.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <errno.h>
 #include <stdlib.h>
 #include <string.h>
 #include <sys/stat.h>
 
 #include <algorithm>
@@ skipping 495 matching lines @@
       v8::String::Utf8Value str(Stringify(isolate, result));
       fwrite(*str, sizeof(**str), str.length(), stdout);
       printf("\n");
     }
   }
   return true;
 }
 
 namespace {
 
-bool IsAbsolutePath(const char* path) {
+std::string ToSTLString(Local<String> v8_str) {
+  String::Utf8Value utf8(v8_str);
+  // Should not be able to fail since the input is a String.
+  CHECK(*utf8);
+  return *utf8;
+}
+
+bool IsAbsolutePath(const std::string& path) {
 #if defined(_WIN32) || defined(_WIN64)
   // TODO(adamk): This is an incorrect approximation, but should
   // work for all our test-running cases.
-  return strchr(path, ':') != nullptr;
+  return path.find(':') != std::string::npos;
 #else
   return path[0] == '/';
 #endif
 }
 
 std::string GetWorkingDirectory() {
 #if defined(_WIN32) || defined(_WIN64)
   char system_buffer[MAX_PATH];
   // TODO(adamk): Support Unicode paths.
   DWORD len = GetCurrentDirectoryA(MAX_PATH, system_buffer);
   CHECK(len > 0);
   return system_buffer;
 #else
   char curdir[PATH_MAX];
   CHECK_NOT_NULL(getcwd(curdir, PATH_MAX));
   return curdir;
 #endif
 }
 
+// Returns the directory part of path, without the trailing '/'.
 std::string DirName(const std::string& path) {
-  DCHECK(IsAbsolutePath(path.c_str()));
-  size_t last_slash = path.find_last_of("/\\");
+  DCHECK(IsAbsolutePath(path));
+  size_t last_slash = path.find_last_of('/');
   DCHECK(last_slash != std::string::npos);
-  return path.substr(0, last_slash + 1);
+  return path.substr(0, last_slash);
+}
+
+std::string EnsureAbsolutePath(const std::string& path,
+                               const std::string& dir_name) {
+  return IsAbsolutePath(path) ? path : dir_name + '/' + path;
 }
 
 MaybeLocal<Module> ResolveModuleCallback(Local<Context> context,
                                          Local<String> specifier,
                                          Local<Module> referrer,
                                          Local<Value> data) {
   Isolate* isolate = context->GetIsolate();
   auto module_map = static_cast<std::map<std::string, Global<Module>>*>(
       External::Cast(*data)->Value());
-  String::Utf8Value specifier_utf8(specifier);
-  CHECK(!IsAbsolutePath(*specifier_utf8));
   Local<String> dir_name = Local<String>::Cast(referrer->GetEmbedderData());
-  std::string absolute_path = *String::Utf8Value(dir_name);
-  absolute_path.append(*specifier_utf8);
+  std::string absolute_path =
+      EnsureAbsolutePath(ToSTLString(specifier), ToSTLString(dir_name));
   auto it = module_map->find(absolute_path);
   if (it != module_map->end()) {
     return it->second.Get(isolate);
   }
   return MaybeLocal<Module>();
 }
 
 }  // anonymous namespace
 
 MaybeLocal<Module> Shell::FetchModuleTree(
     Isolate* isolate, const std::string& file_name,
     std::map<std::string, Global<Module>>* module_map) {
-  DCHECK(IsAbsolutePath(file_name.c_str()));
+  DCHECK(IsAbsolutePath(file_name));
   TryCatch try_catch(isolate);
   try_catch.SetVerbose(true);
   Local<String> source_text = ReadFile(isolate, file_name.c_str());
   if (source_text.IsEmpty()) {
     printf("Error reading '%s'\n", file_name.c_str());
     Shell::Exit(1);
   }
   ScriptOrigin origin(
       String::NewFromUtf8(isolate, file_name.c_str(), NewStringType::kNormal)
           .ToLocalChecked());
   ScriptCompiler::Source source(source_text, origin);
   Local<Module> module;
   if (!ScriptCompiler::CompileModule(isolate, &source).ToLocal(&module)) {
     ReportException(isolate, &try_catch);
     return MaybeLocal<Module>();
   }
   module_map->insert(
       std::make_pair(file_name, Global<Module>(isolate, module)));
 
   std::string dir_name = DirName(file_name);
   module->SetEmbedderData(
       String::NewFromUtf8(isolate, dir_name.c_str(), NewStringType::kNormal)
           .ToLocalChecked());
 
   for (int i = 0, length = module->GetModuleRequestsLength(); i < length; ++i) {
     Local<String> name = module->GetModuleRequest(i);
-    String::Utf8Value utf8_value(name);
-    std::string absolute_path;
-    if (IsAbsolutePath(*utf8_value)) {
-      absolute_path = *utf8_value;
-    } else {
-      absolute_path = dir_name;
-      absolute_path.append(*utf8_value);
-    }
+    std::string absolute_path = EnsureAbsolutePath(ToSTLString(name), dir_name);
     if (!module_map->count(absolute_path)) {
       if (FetchModuleTree(isolate, absolute_path, module_map).IsEmpty()) {
         return MaybeLocal<Module>();
       }
     }
   }
 
   return module;
 }
 
 bool Shell::ExecuteModule(Isolate* isolate, const char* file_name) {
   HandleScope handle_scope(isolate);
 
-  std::string absolute_path;
-  if (IsAbsolutePath(file_name)) {
-    absolute_path = file_name;
-  } else {
-    absolute_path = GetWorkingDirectory();
-    absolute_path.push_back('/');
-    absolute_path.append(file_name);
-  }
+  std::string absolute_path =
+      EnsureAbsolutePath(file_name, GetWorkingDirectory());
+  std::replace(absolute_path.begin(), absolute_path.end(), '\\', '/');
 
   Local<Module> root_module;
   std::map<std::string, Global<Module>> module_map;
   if (!FetchModuleTree(isolate, absolute_path, &module_map)
            .ToLocal(&root_module)) {
     return false;
   }
 
   TryCatch try_catch(isolate);
   try_catch.SetVerbose(true);
@@ skipping 2115 matching lines @@
 }
 
 }  // namespace v8
 
 
 #ifndef GOOGLE3
 int main(int argc, char* argv[]) {
   return v8::Shell::Main(argc, argv);
 }
 #endif