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

correct failed output for direct_state_access.buffers_clear #467

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

okias
Copy link
Contributor

@okias okias commented Jul 17, 2024

correct output for direct_state_access.buffers_clear when failure occurs

For R8* formats, the data is interpreted as non-numeric, so it messes
with the log output. This change ensures the numbers are produced.

Current outcome for R8* formats failure:
The test of ClearNamedBufferData has failed for internalformat . Cleared buffer data is equal to [

After the fix:
The test of ClearNamedBufferData has failed for internalformat . Cleared buffer data is equal to [0], but [5] was expected.

Affects:
KHR-GL46.direct_state_access.buffers_clear

Change-Id: I9e9d771ba129464824df258d5a7066ac797b4c3a

For R8* formats, the data is interpreted as non-numeric, so it messes
with the log output. This change ensures the numbers are produced.

Current outcome for R8* formats failure:
The test of ClearNamedBufferData has failed for internalformat . Cleared buffer data is equal to [

After the fix:
The test of ClearNamedBufferData has failed for internalformat . Cleared buffer data is equal to [0], but [5] was expected.

Affects:
KHR-GL46.direct_state_access.buffers_clear

Change-Id: I9e9d771ba129464824df258d5a7066ac797b4c3a
Signed-off-by: David Heidelberg <[email protected]>
@okias okias force-pushed the 20240717-fix-fail-output branch from 20c9063 to bfc4857 Compare July 17, 2024 18:58
@okias okias marked this pull request as draft July 17, 2024 19:46
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.

1 participant