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

Assorted tweaks #56131

Merged
merged 2 commits into from
Nov 30, 2018
Merged

Assorted tweaks #56131

merged 2 commits into from
Nov 30, 2018

Conversation

ljedrz
Copy link
Contributor

@ljedrz ljedrz commented Nov 21, 2018

  • preallocate VecDeque in Decodable::decode (as it is done with other collections which can do it)
  • add a FIXME to String::from_utf16

r? @RalfJung

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 21, 2018
src/liballoc/rc.rs Outdated Show resolved Hide resolved
@TimNN
Copy link
Contributor

TimNN commented Nov 27, 2018

Ping from triage @RalfJung: It looks like this PR has been updated and is ready for your review again.

@RalfJung
Copy link
Member

RalfJung commented Nov 28, 2018

Thanks @TimNN! @ljedrz: force-pushes do not trigger GH notification emails, that's why I missed the fact that you had updated. In the future, please add a comment when you force-pushed and expect an action by the reviewer :)

@RalfJung
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Nov 28, 2018

📌 Commit 591607d has been approved by RalfJung

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 28, 2018
@ljedrz
Copy link
Contributor Author

ljedrz commented Nov 28, 2018

@RalfJung thanks, I didn't know that 👍.

pietroalbini added a commit to pietroalbini/rust that referenced this pull request Nov 28, 2018
Assorted tweaks

- preallocate `VecDeque` in `Decodable::decode` (as it is done with other collections which can do it)
- add a FIXME to `String::from_utf16`

r? @RalfJung
pietroalbini added a commit to pietroalbini/rust that referenced this pull request Nov 29, 2018
Assorted tweaks

- preallocate `VecDeque` in `Decodable::decode` (as it is done with other collections which can do it)
- add a FIXME to `String::from_utf16`

r? @RalfJung
@RalfJung
Copy link
Member

@bors rollup

kennytm added a commit to kennytm/rust that referenced this pull request Nov 30, 2018
Assorted tweaks

- preallocate `VecDeque` in `Decodable::decode` (as it is done with other collections which can do it)
- add a FIXME to `String::from_utf16`

r? @RalfJung
kennytm added a commit to kennytm/rust that referenced this pull request Nov 30, 2018
Assorted tweaks

- preallocate `VecDeque` in `Decodable::decode` (as it is done with other collections which can do it)
- add a FIXME to `String::from_utf16`

r? @RalfJung
bors added a commit that referenced this pull request Nov 30, 2018
Rollup of 19 pull requests

Successful merges:

 - #55011 (Add libstd Cargo feature "panic_immediate_abort")
 - #55821 (Use sort_by_cached_key when the key function is not trivial/free)
 - #56014 (add test for issue #21335)
 - #56131 (Assorted tweaks)
 - #56214 (Implement chalk unification routines)
 - #56216 (Add TryFrom<&[T]> for [T; $N] where T: Copy)
 - #56268 (Reuse the `P` in `InvocationCollector::fold_{,opt_}expr`.)
 - #56324 (Use raw_entry for more efficient interning)
 - #56336 (Clean up and streamline the pretty-printer)
 - #56337 (Fix const_fn ICE with non-const function pointer)
 - #56339 (Remove not used option)
 - #56341 (Rename conversion util; remove duplicate util in librustc_codegen_llvm.)
 - #56349 (rustc 1.30.0's linker flavor inference is a non-backwards compat change to -Clinker)
 - #56355 (Add inline attributes and add unit to CommonTypes)
 - #56360 (Optimize local linkchecker program)
 - #56364 (Fix panic with outlives in existential type)
 - #56365 (Stabilize self_struct_ctor feature.)
 - #56367 (Moved some feature gate tests to correct location)
 - #56373 (Update books)
@bors bors merged commit 591607d into rust-lang:master Nov 30, 2018
@ljedrz ljedrz deleted the assorted branch November 30, 2018 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants