fixup! src: support configurable snapshot

Co-authored-by: Anna Henningsen <[email protected]>

Author: joyeecheung

src/node.h

@@ -663,7 +663,7 @@ enum Flags : uint64_t {
 enum class SnapshotFlags : uint32_t {
   kDefault = 0,
   // Whether code cache should be generated as part of the snapshot.
-  // Code cache reduce the time spent on compiling functions included
+  // Code cache reduces the time spent on compiling functions included
   // in the snapshot at the expense of a bigger snapshot size and
   // potentially breaking portability of the snapshot.
   kWithoutCodeCache = 1 << 0,
@@ -683,7 +683,7 @@ struct SnapshotConfig {
   // LoadEnvironment() to run a script matching this path. In that case the
   // snapshot is generated as a custom snapshot and the event loop is run, so
   // the snapshot builder can execute asynchronous operations as long as they
-  // are run to completion when the snasphot is taken.
+  // are run to completion when the snapshot is taken.
   std::optional<std::string> builder_script_path;
 };
 

test/embedding/embedtest.cc

@@ -94,7 +94,7 @@ int RunNodeInstance(MultiIsolatePlatform* platform,
         snapshot_config = node::SnapshotConfig{};
       }
       snapshot_config.value().flags = static_cast<node::SnapshotFlags>(
-          static_cast<uint32_t>(snapshot_config.value().flags) &
+          static_cast<uint32_t>(snapshot_config.value().flags) |
           static_cast<uint32_t>(node::SnapshotFlags::kWithoutCodeCache));
     } else if (arg == "--embedder-snapshot-blob") {
       assert(i + 1 < args.size());