From 797402459f7ab55742621eccc337715599252292 Mon Sep 17 00:00:00 2001 From: Daniel Bevenius Date: Sun, 4 Sep 2016 20:49:09 +0200 Subject: [PATCH] src: moving f function call comment The comment about calling the f function seems to have drifted a little. Moving it to be closer to the actual call. --- src/node.cc | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/node.cc b/src/node.cc index d048bd03c8b692..b6ff402724babb 100644 --- a/src/node.cc +++ b/src/node.cc @@ -3388,14 +3388,6 @@ void LoadEnvironment(Environment* env) { CHECK(f_value->IsFunction()); Local f = Local::Cast(f_value); - // 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 - // assigning things to their places. - - // 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 - // like Node's I/O bindings may want to replace 'f' with their own function. - // Add a reference to the global object Local global = env->context()->Global(); @@ -3425,6 +3417,13 @@ void LoadEnvironment(Environment* env) { // (Allows you to set stuff on `global` from anywhere in JavaScript.) global->Set(FIXED_ONE_BYTE_STRING(env->isolate(), "global"), global); + // 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 + // assigning things to their places. + + // 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 + // like Node's I/O bindings may want to replace 'f' with their own function. Local arg = env->process_object(); f->Call(Null(env->isolate()), 1, &arg); }