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.

Sign up for GitHub

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

Jump to bottom

Fix renamer generating duplicate declarations in switch statements #135

Merged
merged 1 commit into from
May 7, 2022
Merged

Fix renamer generating duplicate declarations in switch statements #135

merged 1 commit into from
May 7, 2022

Commits on May 7, 2022

  1. Fix renamer generating duplicate declarations in switch statements 

    The entire switch statement is considered one scope, so we need to
thread through the renamer environment so we don't generate duplicate
definitions.

It turns out the test case I added in laurentlb#132 already hit this -- the
output was not valid GLSL -- but I didn't notice :(
    @jwatzman
    jwatzman committed May 7, 2022
    Configuration menu
    Copy the full SHA
    6722ce4 View commit details
    Browse the repository at this point in the history