GLTFLoader: Remove IIFE closures (#25416)

This commit removes IIFE closures in loadNode() and
loadNodeShallow() in GLTFLoader because they don't look like
really necessary.

examples/jsm/loaders/GLTFLoader.js

@@ -3981,30 +3981,26 @@ class GLTFParser {
 
 		const nodeDef = json.nodes[ nodeIndex ];
 
-		return ( function () {
+		const nodePending = parser._loadNodeShallow( nodeIndex );
 
-			const nodePending = parser._loadNodeShallow( nodeIndex );
+		const childPending = [];
+		const childrenDef = nodeDef.children || [];
 
-			const childPending = [];
-			const childrenDef = nodeDef.children || [];
+		for ( let i = 0, il = childrenDef.length; i < il; i ++ ) {
 
-			for ( let i = 0, il = childrenDef.length; i < il; i ++ ) {
+			childPending.push( parser.getDependency( 'node', childrenDef[ i ] ) );
 
-				childPending.push( parser.getDependency( 'node', childrenDef[ i ] ) );
-
-			}
-
-			const skeletonPending = nodeDef.skin === undefined
-				? Promise.resolve( null )
-				: parser.getDependency( 'skin', nodeDef.skin );
+		}
 
-			return Promise.all( [
-				nodePending,
-				Promise.all( childPending ),
-				skeletonPending
-			] );
+		const skeletonPending = nodeDef.skin === undefined
+			? Promise.resolve( null )
+			: parser.getDependency( 'skin', nodeDef.skin );
 
-		}() ).then( function ( results ) {
+		return Promise.all( [
+			nodePending,
+			Promise.all( childPending ),
+			skeletonPending
+		] ).then( function ( results ) {
 
 			const node = results[ 0 ];
 			const children = results[ 1 ];
@@ -4058,45 +4054,41 @@ class GLTFParser {
 		// reserve node's name before its dependencies, so the root has the intended name.
 		const nodeName = nodeDef.name ? parser.createUniqueName( nodeDef.name ) : '';
 
-		this.nodeCache[ nodeIndex ] = ( function () {
-
-			const pending = [];
-
-			const meshPromise = parser._invokeOne( function ( ext ) {
+		const pending = [];
 
-				return ext.createNodeMesh && ext.createNodeMesh( nodeIndex );
+		const meshPromise = parser._invokeOne( function ( ext ) {
 
-			} );
+			return ext.createNodeMesh && ext.createNodeMesh( nodeIndex );
 
-			if ( meshPromise ) {
+		} );
 
-				pending.push( meshPromise );
+		if ( meshPromise ) {
 
-			}
+			pending.push( meshPromise );
 
-			if ( nodeDef.camera !== undefined ) {
+		}
 
-				pending.push( parser.getDependency( 'camera', nodeDef.camera ).then( function ( camera ) {
+		if ( nodeDef.camera !== undefined ) {
 
-					return parser._getNodeRef( parser.cameraCache, nodeDef.camera, camera );
+			pending.push( parser.getDependency( 'camera', nodeDef.camera ).then( function ( camera ) {
 
-				} ) );
+				return parser._getNodeRef( parser.cameraCache, nodeDef.camera, camera );
 
-			}
+			} ) );
 
-			parser._invokeAll( function ( ext ) {
+		}
 
-				return ext.createNodeAttachment && ext.createNodeAttachment( nodeIndex );
+		parser._invokeAll( function ( ext ) {
 
-			} ).forEach( function ( promise ) {
+			return ext.createNodeAttachment && ext.createNodeAttachment( nodeIndex );
 
-				pending.push( promise );
+		} ).forEach( function ( promise ) {
 
-			} );
+			pending.push( promise );
 
-			return Promise.all( pending );
+		} );
 
-		}() ).then( function ( objects ) {
+		this.nodeCache[ nodeIndex ] = Promise.all( pending ).then( function ( objects ) {
 
 			let node;