Skip to content

Fix PI and TAU in hint_range numeric expressions#119905

Merged
Repiteo merged 1 commit into
godotengine:masterfrom
lupuchard:master
Jul 3, 2026
Merged

Fix PI and TAU in hint_range numeric expressions#119905
Repiteo merged 1 commit into
godotengine:masterfrom
lupuchard:master

Conversation

@lupuchard

Copy link
Copy Markdown
Contributor

What problem(s) does this PR solve?

Additional information

Sets the builtin constant scalars for the other (non-spacial) shaders and fixes and issue where when renaming a constant on it's own (e.g. const float CONSTANT_PI = PI;) it can't be used in constant expressions.

@lupuchard lupuchard requested a review from a team as a code owner May 31, 2026 01:22
@AThousandShips AThousandShips added this to the 4.x milestone Jun 1, 2026

@Chaosus Chaosus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks correct, I would prefer this PR over #119904 since it adds a few other correct lines

@Repiteo Repiteo modified the milestones: 4.x, 4.8 Jul 3, 2026
@Repiteo Repiteo merged commit ec4710b into godotengine:master Jul 3, 2026
20 checks passed
@Repiteo

Repiteo commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Thanks! Congratulations on your first merged contribution! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PI and TAU arent considered valid numeric expressions in hint_range in non-spatial shaders

4 participants