Skip to content

sort functions are clumsy #3265

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
Closed
jesse99 opened this issue Aug 24, 2012 · 1 comment
Closed

sort functions are clumsy #3265

jesse99 opened this issue Aug 24, 2012 · 1 comment
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Milestone
0.5

Comments

@jesse99
Copy link
Contributor

jesse99 commented Aug 24, 2012

The sort module has functions like:
fn merge_sort<T: copy>(le: le<T>, v: [const T]) -> [T]

Usage would be a lot more idiomatic if the function pointer was the last argument.
@brson
Copy link
Contributor

brson commented Aug 24, 2012

Agreed.

@brson brson closed this as completed in 868d101 Oct 25, 2012
bors pushed a commit to rust-lang-ci/rust that referenced this issue May 15, 2021
@topecongiro
Merge pull request rust-lang#3266 from wada314/fix-2973
84702bc 
Fix issue rust-lang#3265
RalfJung pushed a commit to RalfJung/rust that referenced this issue Jan 22, 2024
@bors
Auto merge of rust-lang#3265 - rust-lang:rustup-2024-01-13, r=saethlin
4432da4 
Automatic Rustup
RalfJung pushed a commit to RalfJung/rust that referenced this issue Jan 28, 2024
@bors
Auto merge of rust-lang#3265 - rust-lang:rustup-2024-01-13, r=saethlin
12c0f09 
Automatic Rustup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Milestone
0.5
Development

No branches or pull requests
2 participants
@brson @jesse99