diff options
author | Yash Ladha <yashladhapankajladha123@gmail.com> | 2021-01-30 11:55:32 +0300 |
---|---|---|
committer | Rich Trott <rtrott@gmail.com> | 2021-02-07 01:36:35 +0300 |
commit | d6e94463b3a8c7b7c986c9c2c05c0cfcd5d2768d (patch) | |
tree | 6397d965e08320c5eb2381634c096f248464fe9a /src/env.cc | |
parent | 36cc0ee993a07174f47ea598b6c2c75a66cb052c (diff) |
src: add context for TODO comment in env.cc
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: https://github.com/nodejs/node/pull/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>
Diffstat (limited to 'src/env.cc')
-rw-r--r-- | src/env.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/env.cc b/src/env.cc index ea93574a388..2757319b7b6 100644 --- a/src/env.cc +++ b/src/env.cc @@ -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 |