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

Update comment to more succint for 8/16-bit types #3630

Merged
merged 1 commit into from
Aug 2, 2024

Conversation

ravi688
Copy link
Contributor

@ravi688 ravi688 commented Jun 22, 2024

... that 8/16-bit composites can't be directly constructed.

Earlier the comment looked misleading that composites of 8/16-bit types can't be constructed at all. As per the GL_EXT_shader_16bit_storage extension, they can't be constructed directly, however, can be constructed indirectly.

…ctly constructed

Earlier the comment looked misleading that composites of 8/16-bit types can't be constructed at all.
As per the GL_EXT_shader_16bit_storage extension, they can't be constructed directly, however, can be constructed indirectly.
@ravi688
Copy link
Contributor Author

ravi688 commented Jun 22, 2024

To the maintainer, I'm not sure if the comment updates in this PR are worthwhile - so feel free to reject it.

Also, I've made a comment on #3607 for a possible solution. Suggestions are appreciated for that.

@arcady-lunarg arcady-lunarg merged commit 48154f1 into KhronosGroup:main Aug 2, 2024
22 checks passed
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.

2 participants