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

std: Make vec::from_elem failure-safe #8780

Closed
wants to merge 1 commit into from
Closed

std: Make vec::from_elem failure-safe #8780

wants to merge 1 commit into from

Conversation

brson
Copy link
Contributor

@brson brson commented Aug 27, 2013

No description provided.

bors added a commit that referenced this pull request Aug 27, 2013
@bors
auto merge of #8780 : brson/rust/from_elem, r=thestinger
932d7b9
@bors bors closed this Aug 27, 2013
@thestinger
Copy link
Contributor

@nikomatsakis: the FromIterator/Extendable implementations just use with_capacity + push, but I haven't tested whether the way from_{fn,elem} do it is actually faster

@eddyb eddyb mentioned this pull request Dec 10, 2013
Closed
@eddyb eddyb mentioned this pull request Dec 11, 2013
Merged
eddyb added a commit to eddyb/rust that referenced this pull request Dec 12, 2013
@eddyb
Inline Finallyalizer::drop, allowing LLVM to optimize finally.
331c6ef 
* fixes the vec::from_elem regression caused by rust-lang#8780
* added 5 benchmarks for allocating a 1KB ~[u8] and zeroing it
bors added a commit that referenced this pull request Dec 13, 2013
@bors
auto merge of #10918 : eddyb/rust/inline-finally-dtor, r=thestinger
09bf5de 
* fixes the vec::from_elem regression caused by #8780
* added 5 benchmarks for allocating a 1KB `~[u8]` and zeroing it
* closes #7136
flip1995 pushed a commit to flip1995/rust that referenced this pull request May 5, 2022
@bors
Auto merge of rust-lang#8780 - Alexendoo:init-numbered-field-alias, r…
751fd0d 
…=flip1995

Ignore type aliases in `init_numbered_fields`

changelog: Ignore type aliases in [`init_numbered_fields`]

Fixes rust-lang#8638
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@brson @thestinger @nikomatsakis @bors