Remove unused data structures#47835

Mark-Simulacrum · 2018-01-28T23:34:47Z

Cleanup; as far as I can tell the compiler no longer uses these.

rust-highfive · 2018-01-28T23:34:58Z

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

eddyb · 2018-01-29T00:30:02Z

r? @michaelwoerister

shepmaster · 2018-02-03T01:50:46Z

😻

shepmaster · 2018-02-03T01:51:03Z

Ping from triage, @michaelwoerister !

BatmanAoD · 2018-02-07T17:51:15Z

Ping from triage, @michaelwoerister (also pinged on IRC). @rust-lang/compiler this PR is now 10 days old; if mw does not respond, can a new reviewer be selected?

nikomatsakis · 2018-02-08T16:56:32Z

@bors r+

bors · 2018-02-08T16:56:33Z

📌 Commit caa42e1 has been approved by nikomatsakis

nikomatsakis · 2018-02-08T16:56:37Z

r? @nikomatsakis

…, r=nikomatsakis Remove unused data structures Cleanup; as far as I can tell the compiler no longer uses these.

Rollup of 7 pull requests - Successful merges: #47835, #47854, #48015, #48047, #48051, #48058, #48064 - Failed merges:

Rollup of 12 pull requests - Successful merges: #47835, #47854, #48015, #48047, #48051, #48058, #48064, #47790, #48059, #48078, #48080, #48085 - Failed merges:

…, r=nikomatsakis Remove unused data structures Cleanup; as far as I can tell the compiler no longer uses these.

Rollup of 17 pull requests - Successful merges: #47790, #47835, #47854, #48015, #48047, #48051, #48058, #48059, #48064, #48078, #48080, #48085, #48086, #48090, #48093, #48098, #48101 - Failed merges:

…, r=nikomatsakis Remove unused data structures Cleanup; as far as I can tell the compiler no longer uses these.

Rollup of 16 pull requests - Successful merges: #47790, #47835, #47854, #48015, #48047, #48051, #48058, #48059, #48064, #48078, #48080, #48086, #48098, #48101, #48107, #48100 - Failed merges:

Rollup of 20 pull requests - Successful merges: #47790, #47835, #47854, #48015, #48047, #48051, #48058, #48059, #48064, #48078, #48080, #48086, #48098, #48101, #48107, #48100, #48085, #48120, #48124, #47547 - Failed merges:

bors · 2018-02-10T22:42:11Z

☔ The latest upstream changes (presumably #48113) made this pull request unmergeable. Please resolve the merge conflicts.

michaelwoerister · 2018-02-12T10:09:51Z

Thanks for taking over the review, @nikomatsakis. Back from PTO now.

Mark-Simulacrum added 2 commits January 28, 2018 16:01

Remove unused blake2b implementation

505ef7b

Remove VecCell

caa42e1

rust-highfive assigned alexcrichton Jan 28, 2018

rust-highfive assigned michaelwoerister and unassigned alexcrichton Jan 29, 2018

kennytm added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 29, 2018

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 Feb 8, 2018

rust-highfive assigned nikomatsakis and unassigned michaelwoerister Feb 8, 2018

kennytm added a commit to kennytm/rust that referenced this pull request Feb 8, 2018

Rollup merge of rust-lang#47835 - Mark-Simulacrum:remove-data-structs…

445d699

…, r=nikomatsakis Remove unused data structures Cleanup; as far as I can tell the compiler no longer uses these.

kennytm mentioned this pull request Feb 8, 2018

Rollup of 12 pull requests #48077

Closed

bors added a commit that referenced this pull request Feb 8, 2018

Auto merge of #48077 - kennytm:rollup, r=kennytm

4b3e620

Rollup of 7 pull requests - Successful merges: #47835, #47854, #48015, #48047, #48051, #48058, #48064 - Failed merges:

bors added a commit that referenced this pull request Feb 9, 2018

Auto merge of #48077 - kennytm:rollup, r=kennytm

b29141d

Rollup of 7 pull requests - Successful merges: #47835, #47854, #48015, #48047, #48051, #48058, #48064 - Failed merges:

bors added a commit that referenced this pull request Feb 9, 2018

Auto merge of #48077 - kennytm:rollup, r=kennytm

68df9d6

Rollup of 12 pull requests - Successful merges: #47835, #47854, #48015, #48047, #48051, #48058, #48064, #47790, #48059, #48078, #48080, #48085 - Failed merges:

kennytm added a commit to kennytm/rust that referenced this pull request Feb 9, 2018

Rollup merge of rust-lang#47835 - Mark-Simulacrum:remove-data-structs…

df1c16c

…, r=nikomatsakis Remove unused data structures Cleanup; as far as I can tell the compiler no longer uses these.

kennytm mentioned this pull request Feb 9, 2018

Rollup of 17 pull requests #48104

Closed

kennytm added a commit to kennytm/rust that referenced this pull request Feb 10, 2018

Rollup merge of rust-lang#47835 - Mark-Simulacrum:remove-data-structs…

4f8ea49

…, r=nikomatsakis Remove unused data structures Cleanup; as far as I can tell the compiler no longer uses these.

kennytm mentioned this pull request Feb 10, 2018

Rollup of 20 pull requests #48113

Merged

bors merged commit caa42e1 into rust-lang:master Feb 10, 2018

bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Feb 10, 2018

Mark-Simulacrum deleted the remove-data-structs branch June 8, 2019 13:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Remove unused data structures#47835