Skip to content

Commit

Permalink
WebGLRenderer: Ensure program properties are booleans. (#22244)
Browse files Browse the repository at this point in the history
  • Loading branch information
Mugen87 authored and mrdoob committed Aug 3, 2021
1 parent 75f3cd2 commit f7d4b00
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions src/renderers/WebGLRenderer.js
Original file line number Diff line number Diff line change
Expand Up @@ -1516,10 +1516,10 @@ function WebGLRenderer( parameters = {} ) {
const environment = material.isMeshStandardMaterial ? scene.environment : null;
const encoding = ( _currentRenderTarget === null ) ? _this.outputEncoding : _currentRenderTarget.texture.encoding;
const envMap = ( material.isMeshStandardMaterial ? cubeuvmaps : cubemaps ).get( material.envMap || environment );
const vertexAlphas = material.vertexColors === true && object.geometry && object.geometry.attributes.color && object.geometry.attributes.color.itemSize === 4;
const vertexTangents = object.geometry && object.geometry.attributes.tangent;
const morphTargets = object.geometry && object.geometry.morphAttributes.position;
const morphNormals = object.geometry && object.geometry.morphAttributes.normal;
const vertexAlphas = material.vertexColors === true && !! object.geometry && !! object.geometry.attributes.color && object.geometry.attributes.color.itemSize === 4;
const vertexTangents = !! object.geometry && !! object.geometry.attributes.tangent;
const morphTargets = !! object.geometry && !! object.geometry.morphAttributes.position;
const morphNormals = !! object.geometry && !! object.geometry.morphAttributes.normal;

const materialProperties = properties.get( material );
const lights = currentRenderState.state.lights;
Expand Down
8 changes: 4 additions & 4 deletions src/renderers/webgl/WebGLPrograms.js
Original file line number Diff line number Diff line change
Expand Up @@ -212,9 +212,9 @@ function WebGLPrograms( renderer, cubemaps, cubeuvmaps, extensions, capabilities

combine: material.combine,

vertexTangents: ( material.normalMap && object.geometry && object.geometry.attributes.tangent ),
vertexTangents: ( !! material.normalMap && !! object.geometry && !! object.geometry.attributes.tangent ),
vertexColors: material.vertexColors,
vertexAlphas: material.vertexColors === true && object.geometry && object.geometry.attributes.color && object.geometry.attributes.color.itemSize === 4,
vertexAlphas: material.vertexColors === true && !! object.geometry && !! object.geometry.attributes.color && object.geometry.attributes.color.itemSize === 4,
vertexUvs: !! material.map || !! material.bumpMap || !! material.normalMap || !! material.specularMap || !! material.alphaMap || !! material.emissiveMap || !! material.roughnessMap || !! material.metalnessMap || !! material.clearcoatMap || !! material.clearcoatRoughnessMap || !! material.clearcoatNormalMap || !! material.displacementMap || !! material.transmissionMap || !! material.thicknessMap || !! material.specularIntensityMap || !! material.specularTintMap,
uvsVertexOnly: ! ( !! material.map || !! material.bumpMap || !! material.normalMap || !! material.specularMap || !! material.alphaMap || !! material.emissiveMap || !! material.roughnessMap || !! material.metalnessMap || !! material.clearcoatNormalMap || !! material.transmission || !! material.transmissionMap || !! material.thicknessMap || !! material.specularIntensityMap || !! material.specularTintMap ) && !! material.displacementMap,

Expand All @@ -231,8 +231,8 @@ function WebGLPrograms( renderer, cubemaps, cubeuvmaps, extensions, capabilities
maxBones: maxBones,
useVertexTexture: floatVertexTextures,

morphTargets: object.geometry && object.geometry.morphAttributes.position !== undefined,
morphNormals: object.geometry && object.geometry.morphAttributes.normal !== undefined,
morphTargets: !! object.geometry && !! object.geometry.morphAttributes.position,
morphNormals: !! object.geometry && !! object.geometry.morphAttributes.normal,

numDirLights: lights.directional.length,
numPointLights: lights.point.length,
Expand Down

0 comments on commit f7d4b00

Please sign in to comment.