Revert "src: do proper Maybe usage"

RaisinTen · RaisinTen · commit 4d19a455421c · 2023-04-29T10:34:19.000+05:30
This reverts commit 4bd6caf. Refs: #47588 (comment) Signed-off-by: Darshan Sen <raisinten@gmail.com>
diff --git a/src/json_parser.cc b/src/json_parser.cc
@@ -7,10 +7,7 @@ namespace node {
 using v8::ArrayBuffer;
 using v8::Context;
 using v8::Isolate;
-using v8::Just;
 using v8::Local;
-using v8::Maybe;
-using v8::Nothing;
 using v8::Object;
 using v8::String;
 using v8::Value;
@@ -61,7 +58,8 @@ bool JSONParser::Parse(const std::string& content) {
   return true;
 }
 
-Maybe<std::string> JSONParser::GetTopLevelStringField(std::string_view field) {
+std::optional<std::string> JSONParser::GetTopLevelStringField(
+    std::string_view field) {
   Isolate* isolate = isolate_.get();
   Local<Context> context = context_.Get(isolate);
   Local<Object> content_object = content_.Get(isolate);
@@ -71,17 +69,17 @@ Maybe<std::string> JSONParser::GetTopLevelStringField(std::string_view field) {
       isolate, errors::PrinterTryCatch::kDontPrintSourceLine);
   Local<Value> field_local;
   if (!ToV8Value(context, field, isolate).ToLocal(&field_local)) {
-    return Nothing<std::string>();
+    return {};
   }
   if (!content_object->Get(context, field_local).ToLocal(&value) ||
       !value->IsString()) {
-    return Nothing<std::string>();
+    return {};
   }
   Utf8Value utf8_value(isolate, value);
-  return Just(utf8_value.ToString());
+  return utf8_value.ToString();
 }
 
-Maybe<bool> JSONParser::GetTopLevelBoolField(std::string_view field) {
+std::optional<bool> JSONParser::GetTopLevelBoolField(std::string_view field) {
   Isolate* isolate = isolate_.get();
   Local<Context> context = context_.Get(isolate);
   Local<Object> content_object = content_.Get(isolate);
@@ -92,19 +90,19 @@ Maybe<bool> JSONParser::GetTopLevelBoolField(std::string_view field) {
       isolate, errors::PrinterTryCatch::kDontPrintSourceLine);
   Local<Value> field_local;
   if (!ToV8Value(context, field, isolate).ToLocal(&field_local)) {
-    return Nothing<bool>();
+    return {};
   }
   if (!content_object->Has(context, field_local).To(&has_field)) {
-    return Nothing<bool>();
+    return {};
   }
   if (!has_field) {
-    return Just(false);
+    return false;
   }
   if (!content_object->Get(context, field_local).ToLocal(&value) ||
       !value->IsBoolean()) {
-    return Nothing<bool>();
+    return {};
   }
-  return Just(value->BooleanValue(isolate));
+  return value->BooleanValue(isolate);
 }
 
 }  // namespace node
diff --git a/src/json_parser.h b/src/json_parser.h
@@ -4,6 +4,7 @@
 #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
 
 #include <memory>
+#include <optional>
 #include <string>
 #include "util.h"
 #include "v8.h"
@@ -17,8 +18,8 @@ class JSONParser {
   JSONParser();
   ~JSONParser() {}
   bool Parse(const std::string& content);
-  v8::Maybe<std::string> GetTopLevelStringField(std::string_view field);
-  v8::Maybe<bool> GetTopLevelBoolField(std::string_view field);
+  std::optional<std::string> GetTopLevelStringField(std::string_view field);
+  std::optional<bool> GetTopLevelBoolField(std::string_view field);
 
  private:
   // We might want a lighter-weight JSON parser for this use case. But for now
diff --git a/src/node_sea.cc b/src/node_sea.cc
@@ -161,7 +161,7 @@ std::optional<SeaConfig> ParseSingleExecutableConfig(
   }
 
   result.main_path =
-      parser.GetTopLevelStringField("main").FromMaybe(std::string());
+      parser.GetTopLevelStringField("main").value_or(std::string());
   if (result.main_path.empty()) {
     FPrintF(stderr,
             "\"main\" field of %s is not a non-empty string\n",
@@ -170,23 +170,23 @@ std::optional<SeaConfig> ParseSingleExecutableConfig(
   }
 
   result.output_path =
-      parser.GetTopLevelStringField("output").FromMaybe(std::string());
+      parser.GetTopLevelStringField("output").value_or(std::string());
   if (result.output_path.empty()) {
     FPrintF(stderr,
             "\"output\" field of %s is not a non-empty string\n",
             config_path);
     return std::nullopt;
   }
 
-  bool disable_experimental_sea_warning;
-  if (!parser.GetTopLevelBoolField("disableExperimentalSEAWarning")
-           .To(&disable_experimental_sea_warning)) {
+  std::optional<bool> disable_experimental_sea_warning =
+      parser.GetTopLevelBoolField("disableExperimentalSEAWarning");
+  if (!disable_experimental_sea_warning.has_value()) {
     FPrintF(stderr,
             "\"disableExperimentalSEAWarning\" field of %s is not a Boolean\n",
             config_path);
     return std::nullopt;
   }
-  if (disable_experimental_sea_warning) {
+  if (disable_experimental_sea_warning.value()) {
     result.flags |= SeaFlags::kDisableExperimentalSeaWarning;
   }
 
