Skip to content

Commit

Permalink
Remove manual render state setting
Browse files Browse the repository at this point in the history
Nested render calls now automatically restore the correct render state from the stack.

getRenderState/setRenderState are not part of the public API so they can be safely removed from WebGLRenderer.
  • Loading branch information
Oletus committed Nov 24, 2020
1 parent ad2a514 commit 60defe3
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 14 deletions.
12 changes: 0 additions & 12 deletions src/renderers/WebGLRenderer.js
Original file line number Diff line number Diff line change
Expand Up @@ -1768,18 +1768,6 @@ function WebGLRenderer( parameters ) {

};

this.getRenderState = function () {

return currentRenderState;

};

this.setRenderState = function ( renderState ) {

currentRenderState = renderState;

};

this.getRenderTarget = function () {

return _currentRenderTarget;
Expand Down
2 changes: 0 additions & 2 deletions src/renderers/webgl/WebGLCubeMaps.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,15 +42,13 @@ function WebGLCubeMaps( renderer ) {

const currentRenderList = renderer.getRenderList();
const currentRenderTarget = renderer.getRenderTarget();
const currentRenderState = renderer.getRenderState();

const renderTarget = new WebGLCubeRenderTarget( image.height / 2 );
renderTarget.fromEquirectangularTexture( renderer, texture );
cubemaps.set( texture, renderTarget );

renderer.setRenderTarget( currentRenderTarget );
renderer.setRenderList( currentRenderList );
renderer.setRenderState( currentRenderState );

texture.addEventListener( 'dispose', onTextureDispose );

Expand Down

0 comments on commit 60defe3

Please sign in to comment.