Browse Source

child_process: fix arguments comments

Fixes the arguments comments for execFileSync and other related minor
inconsistencies in commented arguments in the same file.

PR-URL: https://github.com/nodejs/io.js/pull/2161
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
v4.0.0-rc
Roman Reiss 9 years ago
parent
commit
5acad6b163
  1. 10
      lib/child_process.js

10
lib/child_process.js

@ -62,7 +62,7 @@ exports._forkChild = function(fd) {
}; };
function normalizeExecArgs(command /*, options, callback */) { function normalizeExecArgs(command /*, options, callback*/) {
var file, args, options, callback; var file, args, options, callback;
if (typeof arguments[1] === 'function') { if (typeof arguments[1] === 'function') {
@ -98,7 +98,7 @@ function normalizeExecArgs(command /*, options, callback */) {
} }
exports.exec = function(command /*, options, callback */) { exports.exec = function(command /*, options, callback*/) {
var opts = normalizeExecArgs.apply(null, arguments); var opts = normalizeExecArgs.apply(null, arguments);
return exports.execFile(opts.file, return exports.execFile(opts.file,
opts.args, opts.args,
@ -107,7 +107,7 @@ exports.exec = function(command /*, options, callback */) {
}; };
exports.execFile = function(file /* args, options, callback */) { exports.execFile = function(file /*, args, options, callback*/) {
var args, callback; var args, callback;
var options = { var options = {
encoding: 'utf8', encoding: 'utf8',
@ -443,7 +443,7 @@ function checkExecSyncError(ret) {
} }
function execFileSync(/*command, options*/) { function execFileSync(/*command, args, options*/) {
var opts = normalizeSpawnArguments.apply(null, arguments); var opts = normalizeSpawnArguments.apply(null, arguments);
var inheritStderr = !opts.options.stdio; var inheritStderr = !opts.options.stdio;
@ -462,7 +462,7 @@ function execFileSync(/*command, options*/) {
exports.execFileSync = execFileSync; exports.execFileSync = execFileSync;
function execSync(/*comand, options*/) { function execSync(/*command, options*/) {
var opts = normalizeExecArgs.apply(null, arguments); var opts = normalizeExecArgs.apply(null, arguments);
var inheritStderr = opts.options ? !opts.options.stdio : true; var inheritStderr = opts.options ? !opts.options.stdio : true;

Loading…
Cancel
Save