Browse Source

Merge pull request #20567 from Mugen87/dev43

GLTFLoader: Clean up.
Mr.doob 4 years ago
parent
commit
7ac0c2e75c
2 changed files with 4 additions and 4 deletions
  1. 2 2
      examples/js/loaders/GLTFLoader.js
  2. 2 2
      examples/jsm/loaders/GLTFLoader.js

+ 2 - 2
examples/js/loaders/GLTFLoader.js

@@ -755,7 +755,7 @@ THREE.GLTFLoader = ( function () {
 
 			if ( isSupported ) return parser.loadTextureImage( textureIndex, source, loader );
 
-			if ( ! isSupported && json.extensionsRequired && json.extensionsRequired.indexOf( name ) >= 0 ) {
+			if ( json.extensionsRequired && json.extensionsRequired.indexOf( name ) >= 0 ) {
 
 				throw new Error( 'THREE.GLTFLoader: WebP required by asset but unsupported.' );
 
@@ -772,7 +772,7 @@ THREE.GLTFLoader = ( function () {
 
 		if ( ! this.isSupported ) {
 
-			this.isSupported = new Promise( function ( resolve, reject ) {
+			this.isSupported = new Promise( function ( resolve ) {
 
 				var image = new Image();
 

+ 2 - 2
examples/jsm/loaders/GLTFLoader.js

@@ -818,7 +818,7 @@ var GLTFLoader = ( function () {
 
 			if ( isSupported ) return parser.loadTextureImage( textureIndex, source, loader );
 
-			if ( ! isSupported && json.extensionsRequired && json.extensionsRequired.indexOf( name ) >= 0 ) {
+			if ( json.extensionsRequired && json.extensionsRequired.indexOf( name ) >= 0 ) {
 
 				throw new Error( 'THREE.GLTFLoader: WebP required by asset but unsupported.' );
 
@@ -835,7 +835,7 @@ var GLTFLoader = ( function () {
 
 		if ( ! this.isSupported ) {
 
-			this.isSupported = new Promise( function ( resolve, reject ) {
+			this.isSupported = new Promise( function ( resolve ) {
 
 				var image = new Image();