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

CopyTexImage2D references a "CopyPixels" that has been removed from the specification #90

Open
magcius opened this issue May 27, 2023 · 3 comments
Assignees
Labels

Comments

@magcius
Copy link

magcius commented May 27, 2023

https://registry.khronos.org/OpenGL/specs/gl/glspec46.core.pdf#page=240

The image is taken from the framebuffer exactly as if these arguments were passed to CopyPixels (see section 18.3) with argument type set to COLOR, DEPTH, DEPTH_STENCIL, or STENCIL_INDEX, depending on internalformat, stopping after conversion of depth values.

However, section 18.3 does not mention any behavior for CopyPixels. Funnily enough, the changelog says that this text should be moved into this section, but it seems this was forgotten.

Pixel copying - CopyPixels (the comments also applying to CopyTexImage will be moved to section 8.6).

@pdaniell-nv pdaniell-nv added this to the Needs Triage milestone Jun 6, 2023
@pdaniell-nv
Copy link

@oddhack would you mind looking at this one. Thanks.

@oddhack
Copy link
Contributor

oddhack commented Jun 7, 2023

We can put that on the queue for the next time we do a spec update. From the wording I suspect that was a to-do item that was lost track of and rebooting my memory of the pixel/texel deprecated/non-deprecated language will be a project.

@oddhack oddhack added the OpenGL label Jun 7, 2023
@oddhack oddhack modified the milestones: Needs Triage, Needs Action/PR Jun 7, 2023
@magcius
Copy link
Author

magcius commented Jun 7, 2023

I assume it's valid to use the compatibility specification for implementation reference here, since it still has the proper spec language, but I still figured I'd point out the editorial error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants