Skip to content

Add a few problematic std:: functions to our mangling list #1557

Merged
WardBrian merged 2 commits intomasterfrom
fix/stan-3358-stdlib-overloading
Oct 17, 2025
Merged

Add a few problematic std:: functions to our mangling list #1557
WardBrian merged 2 commits intomasterfrom
fix/stan-3358-stdlib-overloading

Conversation

@WardBrian
Copy link
Copy Markdown
Member

Closes stan-dev/stan#3358

There are probably more functions in std:: that don't have concepts/could lead to ambiguity if overloaded by a user in Stan, but these are the ones I spotted based on that issue.

Submission Checklist

  • Run unit tests
  • Documentation
    • If a user-facing facing change was made, the documentation PR is here:
    • OR, no user-facing changes were made

Release notes

Fixed an issue where some function names like min could lead to conflicts with the c++ standard library if they were overloaded by a user.

Copyright and Licensing

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the BSD 3-clause license (https://opensource.org/licenses/BSD-3-Clause)

@WardBrian WardBrian changed the title Fix/stan 3358 stdlib overloading Add a few problematic std:: functions to our mangling list Oct 9, 2025
@WardBrian WardBrian requested a review from SteveBronder October 9, 2025 15:59
@WardBrian WardBrian merged commit 7b9fe98 into master Oct 17, 2025
3 checks passed
@WardBrian WardBrian deleted the fix/stan-3358-stdlib-overloading branch October 17, 2025 16:39
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.

Can't overload binary min/max for real arguments

2 participants