Skip to content

Commit 3f93782

Browse files
MaleDongtargos
MaleDong
authored andcommitted
lib: remove unused exec param
`exec`'s parameters are now deconstructored through `normalizeExecArgs.apply`. We don't need an exclipit parameter anymore (and in fact it's NEVER referred in the code directly), like `spwan` or `spwanSync`. This might be missing. PR-URL: #22274 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: George Adams <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Jon Moss <[email protected]>
1 parent 4d4180b commit 3f93782

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/child_process.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -142,8 +142,8 @@ function normalizeExecArgs(command, options, callback) {
142142
}
143143

144144

145-
exports.exec = function exec(command /* , options, callback */) {
146-
var opts = normalizeExecArgs.apply(null, arguments);
145+
exports.exec = function exec(/* command , options, callback */) {
146+
const opts = normalizeExecArgs.apply(null, arguments);
147147
return exports.execFile(opts.file,
148148
opts.options,
149149
opts.callback);

0 commit comments

Comments
 (0)