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

Use cubeUV_ prefix in cube_uv_reflection_fragment defines #24470

Merged
merged 2 commits into from
Aug 9, 2022

Conversation

CodyJasonBennett
Copy link
Contributor

@CodyJasonBennett CodyJasonBennett commented Aug 8, 2022

Fixed #24469

Description

Renames short defines such as v0 and v1 of cube_uv_reflection_fragment to follow the cubeUV_* pattern and minimize clashing with built-ins such as bsdfs.

@CodyJasonBennett CodyJasonBennett changed the title Use cubeUV_ prefix in defines Use cubeUV_ prefix in cube_uv_reflection_fragment defines Aug 8, 2022
@Mugen87 Mugen87 added this to the r144 milestone Aug 8, 2022
@mrdoob mrdoob merged commit 63103e8 into mrdoob:dev Aug 9, 2022
@mrdoob
Copy link
Owner

mrdoob commented Aug 9, 2022

Thanks!

@CodyJasonBennett CodyJasonBennett deleted the fix/cubeuv-defines-bsdfs branch August 9, 2022 05:58
abernier pushed a commit to abernier/three.js that referenced this pull request Sep 16, 2022
* Use cubeUV_ prefix in defines

* Update cube_uv_reflection_fragment.glsl.js

Co-authored-by: Michael Herzog <[email protected]>
snagy pushed a commit to snagy/three.js-1 that referenced this pull request Sep 21, 2022
* Use cubeUV_ prefix in defines

* Update cube_uv_reflection_fragment.glsl.js

Co-authored-by: Michael Herzog <[email protected]>
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.

Defines in cube_uv_reflection_fragment can break MeshPhongMaterial
4 participants