From 8a41470c85536119a6dbeaf4b40dc9ebc4c453c4 Mon Sep 17 00:00:00 2001
From: MaleDong <maledong_github@outlook.com>
Date: Sun, 12 Aug 2018 11:22:22 +0800
Subject: [PATCH] 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: https://github.com/nodejs/node/pull/22274
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: George Adams <george.adams@uk.ibm.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
---
 lib/child_process.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/child_process.js b/lib/child_process.js
index 8460a732d9bf43..6cce1b09e5654b 100644
--- a/lib/child_process.js
+++ b/lib/child_process.js
@@ -142,8 +142,8 @@ function normalizeExecArgs(command, options, callback) {
 }
 
 
-exports.exec = function exec(command /* , options, callback */) {
-  var opts = normalizeExecArgs.apply(null, arguments);
+exports.exec = function exec(/* command , options, callback */) {
+  const opts = normalizeExecArgs.apply(null, arguments);
   return exports.execFile(opts.file,
                           opts.options,
                           opts.callback);