Skip to content

Commit

Permalink
src: add context for TODO comment in env.cc
Browse files Browse the repository at this point in the history
Add more context around the TODO change required for achieving the
task. When destructuring the isolate and environment_vars from the
environment object, it is leading to recursive dependency and thus not
able to refactor it in a better way.

PR-URL: #37140
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Pooja D P <Pooja.D.P@ibm.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
  • Loading branch information
yashLadha authored and Trott committed Feb 6, 2021
1 parent 36cc0ee commit d6e9446
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/env.cc
Original file line number Diff line number Diff line change
Expand Up @@ -368,7 +368,8 @@ Environment::Environment(IsolateData* isolate_data,

set_env_vars(per_process::system_environment);
// TODO(joyeecheung): pass Isolate* and env_vars to it instead of the entire
// env
// env, when the recursive dependency inclusion in "debug-utils.h" is
// resolved.
enabled_debug_list_.Parse(this);

// We create new copies of the per-Environment option sets, so that it is
Expand Down

0 comments on commit d6e9446

Please sign in to comment.