Skip to content

Commit

Permalink
fixup! src: drain platform tasks before creating startup snapshot
Browse files Browse the repository at this point in the history
  • Loading branch information
legendecas committed Jan 6, 2025
1 parent 4642ec5 commit c1a5d20
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions src/node_snapshotable.cc
Original file line number Diff line number Diff line change
Expand Up @@ -992,10 +992,6 @@ ExitCode BuildSnapshotWithoutCodeCache(
// example, a WeakRef may schedule an per-isolate platform task as a GC
// root, and referencing an object in a context, causing an assertion in
// the snapshot creator.

// FIXME(joyeecheung): right now running the loop in the snapshot
// builder might introduce inconsistencies in JS land that need to
// be synchronized again after snapshot restoration.
ExitCode exit_code =
SpinEventLoopInternal(env).FromMaybe(ExitCode::kGenericUserError);
if (exit_code != ExitCode::kNoFailure) {
Expand Down

0 comments on commit c1a5d20

Please sign in to comment.