Skip to content

Mention std::clamp in explanatory comment instead of min/max #256

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
Merged
merged 1 commit into from
Dec 20, 2021
Merged

Mention std::clamp in explanatory comment instead of min/max #256

merged 1 commit into from
Dec 20, 2021

Conversation

ralphtheninja
Copy link

@ralphtheninja ralphtheninja commented Dec 17, 2021 •
edited
Loading

I suspect the example code used std::min and std::max and was later changed to std::clamp.

@Overv Overv merged commit 4013f80 into Overv:master Dec 20, 2021
@Overv
Copy link
Owner

Overv commented Dec 20, 2021

You're right, thanks!

@ralphtheninja ralphtheninja deleted the std-clamp branch December 21, 2021 09:58
Overv added a commit that referenced this pull request Mar 8, 2023
@Overv
Merge pull request #256 from ralphtheninja/std-clamp
e1292ac 
Mention std::clamp in explanatory comment instead of min/max
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ralphtheninja @Overv