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

glObjectPtrLabel inconsistensies #446

Open
frederikja163 opened this issue Nov 21, 2020 · 4 comments
Open

glObjectPtrLabel inconsistensies #446

frederikja163 opened this issue Nov 21, 2020 · 4 comments
Assignees

Comments

@frederikja163
Copy link
Contributor

The methods glObjectPtrLabel and glObjectPtrLabelKHR are inconsistent with whats defined in the documentation on several points.

https://www.khronos.org/registry/OpenGL-Refpages/gl4/html/glObjectPtrLabel.xhtml

ptr parameter is defined as const void* in the spec while its defined as void * (without the const) in the documentation.
ptr is also described as being a sync object in the documentation while its type is still a void*

@pdaniell-nv
Copy link
Contributor

ptr parameter is defined as const void* in the spec while its defined as void * (without the const) in the documentation.
ptr is also described as being a sync object in the documentation while its type is still a void*

Can you be more specific what you mean by "spec" and "documentation" here. On page 558 of the OpenGL 4.6 spec (https://www.khronos.org/registry/OpenGL/specs/gl/glspec46.core.pdf) ptr is defined as "void *".

@frederikja163
Copy link
Contributor Author

I was refering to the XML file, in there its defined as const void *

@Perksey
Copy link
Contributor

Perksey commented Jan 25, 2021

That looks to be an error in the XML file then, but I fear it won't be able to be corrected for compatibility reasons.

@frederikja163
Copy link
Contributor Author

We just started working Documentation into our bindings generator, for this we are using the XML files talked about in this issue. We have found many other issues similar to this one and are currently working on a tool to tell us where the inconsistencies are. This should allow us to fix all the inconsistencies. 🤞

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

No branches or pull requests

4 participants
@pdaniell-nv @Perksey @frederikja163 and others