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

Fix Hello Triangle Breathing example not working properly #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Mcgode
Copy link

@Mcgode Mcgode commented May 7, 2023

time is declared on the stack, but was accessed through a pointer after being unscoped when calling rpsRenderGraphUpdate().
It resulted in receiving an invalid value in DrawTriangleBreathingCb, breaking the demo.

This PR fixes the issue by moving the local variable declaration to the same scope as rpsRenderGraphUpdate().

@FlorianHerickAMD
Copy link
Collaborator

Thank you for reporting this. We will address this either as part of a small maintenance update or the next version.

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