Skip to content

Conversation

@DyXel
Copy link
Contributor

@DyXel DyXel commented Jun 4, 2025

  • GL_PRIMITIVE_RESTART is not a valid enum for glEnable&co on GLES 3.20
  • GL_CONTEXT_PROFILE_MASK is not a valid enum for glGetIntegerv on GLES 3.20

This ended up pushing GL_INVALID_ENUM to the stack when the aforementioned GL
functions were called, and as these functions are not checked for errors, they
would end up appearing later, possibly in user's code under unrelated GL functions.

(Ask me how I know they could appear in user's code 😅)
Note that I am going off the OpenGL ES 3.2 Reference Pages, which appears to be correct in this instance.

* GL_PRIMITIVE_RESTART is not a valid enum for glEnable&co on GLES 3.20
* GL_CONTEXT_PROFILE_MASK is not a valid enum for glGetIntegerv on GLES 3.20

This ended up pushing GL_INVALID_ENUM to the stack when the aforementioned GL
functions were called, and as these functions are not checked for errors, they
would end up appearing later, possibly in user's code.
ocornut pushed a commit that referenced this pull request Jun 4, 2025
…ch would push a GL error. (#8664)

* GL_PRIMITIVE_RESTART is not a valid enum for glEnable&co on GLES 3.20
* GL_CONTEXT_PROFILE_MASK is not a valid enum for glGetIntegerv on GLES 3.20
@ocornut
Copy link
Owner

ocornut commented Jun 4, 2025

Thank you Dylam.
This has been merged and pushed as 201899b.

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.

2 participants