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.

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

proposal: builtin: add min() and max() functions as builtins #54182

Closed
p7r0x7 opened this issue Aug 1, 2022 · 2 comments
Closed

proposal: builtin: add min() and max() functions as builtins #54182

p7r0x7 opened this issue Aug 1, 2022 · 2 comments
Labels
FrozenDueToAge Proposal
Milestone
Proposal

Comments

@p7r0x7
Copy link

p7r0x7 commented Aug 1, 2022

I've noticed a lot of packages (especially standard ones) use these functions when they probably should be sharing the function globally rather than building it for each individual package that uses them. Yes, I'm aware this would break a lot of existing code so this can't be in Go1, but maybe it can be added somewhere as standard?
@p7r0x7 p7r0x7 added the Proposal label Aug 1, 2022
@gopherbot gopherbot added this to the Proposal milestone Aug 1, 2022
@seankhliao
Copy link
Member

No need for them to be builtins, they can be library functions

@seankhliao
Copy link
Member

Duplicate of #50019

@seankhliao seankhliao marked this as a duplicate of #50019 Aug 1, 2022
@seankhliao seankhliao closed this as not planned Won't fix, can't repro, duplicate, stale Aug 1, 2022
@golang golang locked and limited conversation to collaborators Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge Proposal
Projects
None yet
Milestone
Proposal
Development

No branches or pull requests
3 participants
@gopherbot @seankhliao @p7r0x7