read too many vertices

timknip · Jan 27, 2010 · 9143014 · 9143014
1 parent 92a8c1a
commit 9143014
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/README b/README
@@ -7,6 +7,12 @@ if you have experience with lib3ds, many objects will look familiar (Lib3dsMesh
 At the moment only meshes are read: vertices, faces and texels. 
 No materials etc. yet.
 
+NOTE: 
+I didn't actually test this, so... one thing: Endian. 
+Default is now little endian, You might want to pass 'true' as
+first argument to the BinaryParser constructor to setup big endian.
+
+
 TODO
 =======================================================
 Much :-)

diff --git a/index.html b/index.html
@@ -25,7 +25,7 @@
 			}
 		</script>
 	</head>
-	<body onload="loadFile('MIG-21.3ds')">
+	<body onload="loadFile('bobafett.3ds')">
 		<div id="dbg"></div>
 	</body>
 </html>
diff --git a/js/lib3ds.js b/js/lib3ds.js
@@ -305,13 +305,11 @@ with ({p:Lib3ds.prototype}) {
 				case POINT_ARRAY:
 					mesh.points = this.readWord(data);
 					mesh.pointL = [];
-					this.log (" -> #points: " + mesh.points);
-					for (i = 0; i < mesh.points; i++) {
+					this.log (" -> #points: " + mesh.points + " " + this.position);
+					for (i = 0; i < mesh.points-1; i++) {
 						var vec = [];
 						for (j = 0; j < 3; j++) {
 							vec.push(this.readFloat(data));
-							vec.push(this.readFloat(data));
-							vec.push(this.readFloat(data));
 						}
 						mesh.pointL.push(vec);
 					}
@@ -432,9 +430,13 @@ with ({p:Lib3ds.prototype}) {
 	}
 
 	p.readFloat = function(data) {
+		try {
 		var v = this.parser.toFloat(data.substr(this.position, 4));
 		this.position += 4;
 		return v;
+		} catch(e) {
+			this.log("" + e + " " + this.position + " " + data.length);
+		}
 	}
 
 	p.readInt = function(data) {