refactor: move functions to a higher scope

Author: headlessNode

lib/node_modules/@stdlib/_tools/eslint/rules/vars-order/lib/main.js

@@ -82,47 +82,34 @@ function main( context ) {
 	}
 
 	/**
-	* Checks whether the variable declarations inside of the supplied node are ordered.
+	* Sorts the variable declarations by name length.
 	*
 	* @private
-	* @param {ASTNode} node - node to examine
-	* @returns {void}
+	* @param {Object} a - input object
+	* @param {Object} b - comparison object
+	* @returns {number} number indicating sort order
 	*/
-	function validate( node ) {
-		var prevLength;
-		var body;
-		var elem;
-		var name;
-		var i;
-
-		body = node.body.body;
-		prevLength = null;
-		for ( i = 0; i < body.length; i++ ) {
-			elem = body[ i ];
-			if ( elem.type === 'VariableDeclaration' && elem.kind === 'var' ) {
-				name = elem.declarations[ 0 ].id.name;
-				if ( prevLength && !fun( name.length, prevLength ) ) {
-					return report( 'Variable declarations inside of function are not ordered by length (in '+ order +' order)', node.loc );
-				}
-				prevLength = name.length;
-			}
+	function sortVars( a, b ) {
+		if ( fun( a.name.length, b.name.length ) ) {
+			return 1;
 		}
+		return -1;
+	}
 
-		/**
-		* Reports the error message.
-		*
-		* @private
-		* @param {string} msg - error message
-		* @param {Object} loc - lines of code (object with `start` and `end` properties)
-		*/
-		function report( msg, loc ) {
-			context.report({
-				'node': null,
-				'message': msg,
-				'loc': loc,
-				'fix': fix
-			});
-		}
+	/**
+	* Reports the error message.
+	*
+	* @private
+	* @param {string} msg - error message
+	* @param {ASTNode} node - node to fix
+	*/
+	function report( msg, node ) {
+		context.report({
+			'node': null,
+			'message': msg,
+			'loc': node.loc,
+			'fix': fix
+		});
 
 		/**
 		* Fixes the lint error by reordering the variable declarations inside of the function.
@@ -138,10 +125,13 @@ function main( context ) {
 			var endRange;
 			var source;
 			var elem;
+			var body;
+			var i;
 			var j;
 
 			declarations = [];
 			replacingText = '';
+			body = node.body.body;
 			source = context.getSourceCode();
 
 			for ( i = 0; i < body.length; i++ ) {
@@ -172,20 +162,33 @@ function main( context ) {
 			}
 
 			return fixer.replaceTextRange( [ startRange, endRange ], replacingText ); // eslint-disable-line max-len
+		}
+	}
 
-			/**
-			* Sorts the variable declarations by name length.
-			*
-			* @private
-			* @param {Object} a - input object
-			* @param {Object} b - comparison object
-			* @returns {number} number indicating sort order
-			*/
-			function sortVars( a, b ) {
-				if ( fun( a.name.length, b.name.length ) ) {
-					return 1;
+	/**
+	* Checks whether the variable declarations inside of the supplied node are ordered.
+	*
+	* @private
+	* @param {ASTNode} node - node to examine
+	* @returns {void}
+	*/
+	function validate( node ) {
+		var prevLength;
+		var body;
+		var elem;
+		var name;
+		var i;
+
+		body = node.body.body;
+		prevLength = null;
+		for ( i = 0; i < body.length; i++ ) {
+			elem = body[ i ];
+			if ( elem.type === 'VariableDeclaration' && elem.kind === 'var' ) {
+				name = elem.declarations[ 0 ].id.name;
+				if ( prevLength && !fun( name.length, prevLength ) ) {
+					return report( 'Variable declarations inside of function are not ordered by length (in '+ order +' order)', node );
 				}
-				return -1;
+				prevLength = name.length;
 			}
 		}
 	}