Closed acolwell closed 1 year ago
Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. Additionally, make sure you've done all of these things:
What type of PR is this? (Check one of the boxes below)
What does this pull request do?
Specifying the MathUtils namespace to resolve the ambiguity caused by the addition of std::clamp() in C++17.
This fixes a build error on Windows with a freshly installed MSYS2 where g++ uses c++17 by default.
Have you tested your changes (if applicable)? If so, how?
Yes. The code that uses MathUtils::clamp() properly builds after this change.
Thanks for submitting a pull request! Please provide enough information so that others can review your pull request. Additionally, make sure you've done all of these things:
PR Description
What type of PR is this? (Check one of the boxes below)
What does this pull request do?
Specifying the MathUtils namespace to resolve the ambiguity caused by the addition of std::clamp() in C++17.
This fixes a build error on Windows with a freshly installed MSYS2 where g++ uses c++17 by default.
Have you tested your changes (if applicable)? If so, how?
Yes. The code that uses MathUtils::clamp() properly builds after this change.