Browse Source

src: update f function call comment

Commit a01e8bc (src: moving f function call comment, 2016-09-04) moved
a comment to be closer to the place in the code it was describing.
While working on this there was feedback on the comment
itself:
https://github.com/nodejs/node/commit/
a01e8bcf189bd598d496f347f60007bc2211e528#comments

This PR includes the suggestions in the above comments.

PR-URL: https://github.com/nodejs/node/pull/8416
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Franziska Hinkelmann <franzih@chromium.org>
Reviewed-By: James M Snell <jasnell@gmail.com>
v6.x
Daniel Bevenius 8 years ago
committed by Jeremiah Senkpiel
parent
commit
8cbbb47e39
  1. 6
      src/node.cc

6
src/node.cc

@ -3498,11 +3498,11 @@ void LoadEnvironment(Environment* env) {
global->Set(FIXED_ONE_BYTE_STRING(env->isolate(), "global"), global); global->Set(FIXED_ONE_BYTE_STRING(env->isolate(), "global"), global);
// Now we call 'f' with the 'process' variable that we've built up with // Now we call 'f' with the 'process' variable that we've built up with
// all our bindings. Inside bootstrap_node.js we'll take care of // all our bindings. Inside bootstrap_node.js and internal/process we'll
// assigning things to their places. // take care of assigning things to their places.
// We start the process this way in order to be more modular. Developers // We start the process this way in order to be more modular. Developers
// who do not like how bootstrap_node.js setups the module system but do // who do not like how bootstrap_node.js sets up the module system but do
// like Node's I/O bindings may want to replace 'f' with their own function. // like Node's I/O bindings may want to replace 'f' with their own function.
Local<Value> arg = env->process_object(); Local<Value> arg = env->process_object();
f->Call(Null(env->isolate()), 1, &arg); f->Call(Null(env->isolate()), 1, &arg);

Loading…
Cancel
Save