Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove call to checkFramebufferStatus #23770

Merged
merged 1 commit into from
Mar 23, 2022

Conversation

greggman
Copy link
Contributor

This is a perf hit and AFAIK there's not much reason to do the check
except to give the user an error message that the browser itself
would print.

No other place in the code calls checkFramebufferStatus so not sure
why this is special. If the framebuffer is not complete you'd be
getting rendering warnings from the browser as well.

Related issue: #XXXX

Description

A clear and concise description of what the problem was and how this pull request solves it.

This contribution is funded by Example.

This is a perf hit and AFAIK there's not much reason to do the check
except to give the user an error message that the browser itself
would print.

No other place in the code calls checkFramebufferStatus so not sure
why this is special. If the framebuffer is not complete you'd be
getting rendering warnings from the browser as well.
@gkjohnson
Copy link
Collaborator

Is it worth guarding this behind a debug flag like we do shader error reporting?

https://threejs.org/docs/?q=renderer#api/en/renderers/WebGLRenderer.debug

@greggman
Copy link
Contributor Author

I'd say no unless you're going to check all WebGL errors throughout the code. I'm not sure what makes this one special.

@mrdoob mrdoob added this to the r139 milestone Mar 23, 2022
@mrdoob mrdoob merged commit 6f1caec into mrdoob:dev Mar 23, 2022
@mrdoob
Copy link
Owner

mrdoob commented Mar 23, 2022

Thanks!

abernier pushed a commit to abernier/three.js that referenced this pull request Sep 16, 2022
This is a perf hit and AFAIK there's not much reason to do the check
except to give the user an error message that the browser itself
would print.

No other place in the code calls checkFramebufferStatus so not sure
why this is special. If the framebuffer is not complete you'd be
getting rendering warnings from the browser as well.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants