Browse Source

Geometry: Simplified to just merge() and mergeMesh().

Mr.doob 10 years ago
parent
commit
ec7d25e18b
1 changed files with 17 additions and 24 deletions
  1. 17 24
      src/core/Geometry.js

+ 17 - 24
src/core/Geometry.js

@@ -161,7 +161,7 @@ THREE.Geometry.prototype = {
 			}
 
 		}
-		
+
 		this.computeFaceNormals();
 
 		if ( geometry.boundingBox !== null ) {
@@ -557,32 +557,10 @@ THREE.Geometry.prototype = {
 	},
 
 	merge: function ( geometry, matrix, materialIndexOffset ) {
-	
-		console.warn( 'THREE.Geometry.merge() has been renamed to THREE.Geometry.mergeGeometry().' );
-		this.mergeGeometry( geometry, matrix, materialIndexOffset );
-	
-	},
-
-	mergeMesh: function ( mesh ) {
-	
-		if ( mesh instanceof THREE.Mesh === false ) {
-		
-			console.error( 'THREE.Geometry.mergeMesh(): mesh not an instance of THREE.Mesh.', mesh );
-			return;
-		
-		}
-
-		mesh.matrixAutoUpdate && mesh.updateMatrix();
-		
-		this.mergeGeometry( mesh.geometry, mesh.matrix );
-	
-	},
-
-	mergeGeometry: function ( geometry, matrix, materialIndexOffset ) {
 
 		if ( geometry instanceof THREE.Geometry === false ) {
 
-			console.error( 'THREE.Geometry.mergeGeometry(): geometry not an instance of THREE.Geometry.', geometry );
+			console.error( 'THREE.Geometry.merge(): geometry not an instance of THREE.Geometry.', geometry );
 			return;
 
 		}
@@ -688,6 +666,21 @@ THREE.Geometry.prototype = {
 
 	},
 
+	mergeMesh: function ( mesh ) {
+
+		if ( mesh instanceof THREE.Mesh === false ) {
+
+			console.error( 'THREE.Geometry.mergeMesh(): mesh not an instance of THREE.Mesh.', mesh );
+			return;
+
+		}
+
+		mesh.matrixAutoUpdate && mesh.updateMatrix();
+
+		this.merge( mesh.geometry, mesh.matrix );
+
+	},
+
 	/*
 	 * Checks for duplicate vertices with hashmap.
 	 * Duplicated vertices are removed