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

GLES2: Revert wrong extension checks from #26928 and #26932 #26936

Merged
merged 3 commits into from
Mar 11, 2019

Conversation

akien-mga
Copy link
Member

texture2DLodEXT and textureCubeLodEXT are only for the fragment shader with
https://www.khronos.org/registry/OpenGL/extensions/EXT/EXT_shader_texture_lod.txt
In the vertex shader, texture2DLod and textureCubeLod are built-in.

So my "fixes" were actually bogus. Added a comment to clarify.

To avoid reintroducing bugs as I did in godotengine#26928 and godotengine#26932.

texture2DLodEXT and textureCubeLodEXT are only for the fragment shader with
https://www.khronos.org/registry/OpenGL/extensions/EXT/EXT_shader_texture_lod.txt
In the vertex shader, texture2DLod and textureCubeLod are built-in.
@akien-mga akien-mga added this to the 3.1 milestone Mar 11, 2019
@akien-mga akien-mga requested a review from reduz as a code owner March 11, 2019 22:21
@akien-mga akien-mga changed the title Gles2 revert texture2 d lod ext GLES2: Revert wrong extension checks from #26928 and #26932 Mar 11, 2019
@akien-mga akien-mga merged commit 7452f5c into godotengine:master Mar 11, 2019
@akien-mga akien-mga deleted the gles2-revert-texture2DLodEXT branch March 11, 2019 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant