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.

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

Remove RealExt and clean up some of stc::num::cmath #11504

Merged
merged 2 commits into from
Jan 13, 2014

Conversation

brendanzab
Copy link
Member

The gamma and bessel functions are of little utility outside a small specialized subset of use cases, and so they don't really make sense for inclusion in the standard library. The only reason they were included in the first place was to mirror libm, which is not a very good justification. If people need them for their own projects then they can make their own bindings to libm (which isn't too hard).

These functions are of little utility outside a small subset of use cases. If people need them for their own projects then they can use their own bindings for libm (which aren't hard to make).
bors added a commit that referenced this pull request Jan 13, 2014
The gamma and bessel functions are of little utility outside a small specialized subset of use cases, and so they don't really make sense for inclusion in the standard library. The only reason they were included in the first place was to mirror libm, which is not a very good justification. If people need them for their own projects then they can make their own bindings to libm (which isn't too hard).
@brendanzab
Copy link
Member Author

cc. #10387

@bors bors closed this Jan 13, 2014
@bors bors merged commit cd248e2 into rust-lang:master Jan 13, 2014
@brendanzab brendanzab deleted the std-num-cleanups branch January 13, 2014 01:30
flip1995 pushed a commit to flip1995/rust that referenced this pull request Sep 25, 2023
…dnet

Ignore closures for some type lints

Fixes rust-lang#11417

`hir_ty_to_ty` is used in a couple of the `!is_local` lints, which doesn't play nicely inside bodies

changelog: none
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.

3 participants