Skip to content

Change ~[T] to Vec<T> in librustc #12716

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 4 commits into from
Closed

Change ~[T] to Vec<T> in librustc #12716

wants to merge 4 commits into from

Conversation

pcwalton
Copy link
Contributor

@pcwalton pcwalton commented Mar 5, 2014

@pnkfelix
Copy link
Member

pnkfelix commented Mar 6, 2014

Okay, r=me after you address comments above from @thestinger and myself.

pnkfelix added a commit to pnkfelix/rust that referenced this pull request Mar 8, 2014
@pnkfelix
Incorporated review feedback atop pcwalton's original patches.
f978c77 
(Original PR was rust-lang#12716; feedback was provided by thestinger and me.)
@pnkfelix pnkfelix mentioned this pull request Mar 8, 2014
Closed
@pnkfelix
Copy link
Member

pnkfelix commented Mar 8, 2014

closing in favor of PR #12768 (which is a rebased version of this with an extra commit on top that adds the changes based on review feedback).

@pnkfelix pnkfelix closed this Mar 8, 2014
bors added a commit that referenced this pull request Mar 8, 2014
@bors
auto merge of #12768 : pnkfelix/rust/fsk-devecing, r=pnkfelix
0017056 
Change `~[T]` to Vec<T> in librustc.  Rebased and amended version of PR #12716.

Original author (or perhaps I should say meta-author) was @pcwalton, as is reflected in the commits.

I clean up!  :)
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
@bors
Auto merge of rust-lang#12716 - davidlattimore:fix-publish-crate-name…
09abd76 
…s, r=lnicola

publish workflow: Fix names for existing crates

Attempting to publish for example ra_ap_text-edit when ra_ap_text_edit
has already been published is rejected by crates.io.
matthiaskrgr pushed a commit to matthiaskrgr/rust that referenced this pull request Aug 2, 2022
@bors
Auto merge of rust-lang#12900 - pksunkara:master, r=lnicola
af2b806 
publish: Use cargo ws rename to rename crates

Follow up for rust-lang#12716
# for free to join this conversation on GitHub. Already have an account? # 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.
3 participants
@pcwalton @pnkfelix @thestinger