Refactor to avoid use of variable in parent scope

kgryte · kgryte · commit fad1635565ce · 2017-12-13T20:21:28.000-08:00
diff --git a/lib/node_modules/@stdlib/_tools/pkgs/deps/bin/cli b/lib/node_modules/@stdlib/_tools/pkgs/deps/bin/cli
@@ -54,7 +54,6 @@ function done( error, results ) {
 function main() {
 	var flags;
 	var args;
-	var pkgs;
 	var opts;
 	var cli;
 	var sep;
@@ -98,13 +97,12 @@ function main() {
 	if ( !process.stdin.isTTY ) {
 		return stdin( onData );
 	}
-	pkgs = args.slice();
-	if ( pkgs.length === 0 ) {
+	if ( args.length === 0 ) {
 		err = new Error( 'no input. Must provide a list of packages as command-line arguments or via `stdin`.' );
 		process.exitCode = 1;
 		return console.error( 'Error: %s', err.message ); // eslint-disable-line no-console
 	}
-	process.nextTick( next );
+	return next( args.slice() );
 
 	/**
 	* Callback invoked after attempting to read data from `stdin`.
@@ -115,6 +113,7 @@ function main() {
 	* @returns {void}
 	*/
 	function onData( error, data ) {
+		var pkgs;
 		var i;
 		if ( error ) {
 			process.exitCode = 1;
@@ -134,15 +133,16 @@ function main() {
 				pkgs.push( data[ i ] );
 			}
 		}
-		next();
+		next( pkgs );
 	} // end FUNCTION onData()
 
 	/**
 	* Callback invoked once ready to resolve dependencies.
 	*
 	* @private
+	* @param {StringArray} pkgs - list of packages
 	*/
-	function next() {
+	function next( pkgs ) {
 		find( pkgs, opts, done );
 	} // end FUNCTION next()
 } // end FUNCTION main()
