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

Remove a few unused objects #69059

Merged
merged 1 commit into from
Feb 12, 2020
Merged

Remove a few unused objects #69059

merged 1 commit into from
Feb 12, 2020

Conversation

ljedrz
Copy link
Contributor

@ljedrz ljedrz commented Feb 11, 2020

As far as I can tell, these won't be missed:

  • infer::region_constraints::ConstraintInfo
  • driver::DefaultCallbacks
  • hir::intravisit::ParDeepVisitor

@rust-highfive
Copy link
Collaborator

r? @ecstatic-morse

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

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Feb 11, 2020
@ljedrz
Copy link
Contributor Author

ljedrz commented Feb 11, 2020

Sooo I guess I'll remove that bit again shortly ^^.

bors added a commit to rust-lang/rust-clippy that referenced this pull request Feb 11, 2020
Impl DefaultCallbacks on our side

cc rust-lang/rust#69059 (comment)

changelog: none
@ecstatic-morse
Copy link
Contributor

ecstatic-morse commented Feb 11, 2020

hir::intravisit::ParDeepVisitor seems like it could be useful in the future, even if it is not currently used. I don't think it should be removed since there's no significant benefit to doing so cc @Zoxc.

ConstraintInfo can go though.

bors added a commit to rust-lang/rust-clippy that referenced this pull request Feb 11, 2020
Impl DefaultCallbacks on our side

cc rust-lang/rust#69059 (comment)

changelog: none
@Zoxc
Copy link
Contributor

Zoxc commented Feb 12, 2020

I do have a PR open which make use of ParDeepVisitor =P

@ljedrz
Copy link
Contributor Author

ljedrz commented Feb 12, 2020

Thanks for the feedback, I'll update the PR shortly.

@ljedrz
Copy link
Contributor Author

ljedrz commented Feb 12, 2020

Done.

@Dylan-DPC-zz
Copy link

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Feb 12, 2020

📌 Commit d8544ce has been approved by Dylan-DPC

@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 Feb 12, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Feb 12, 2020
Remove a few unused objects

As far as I can tell, these won't be missed:

- `infer::region_constraints::ConstraintInfo`
- `driver::DefaultCallbacks`
- ~~`hir::intravisit::ParDeepVisitor`~~
bors added a commit that referenced this pull request Feb 12, 2020
Rollup of 8 pull requests

Successful merges:

 - #67585 (Improve `char::is_ascii_*` codegen)
 - #68914 (Speed up `SipHasher128`.)
 - #68994 (rustbuild: include channel in sanitizers installed name)
 - #69032 (ICE in nightly-2020-02-08: handle TerminatorKind::Yield in librustc_mir::transform::promote_consts::Validator method)
 - #69034 (parser: Remove `Parser::prev_token_kind`)
 - #69042 (Remove backtrace header text)
 - #69059 (Remove a few unused objects)
 - #69089 (Properly use the darwin archive format on Apple targets)

Failed merges:

r? @ghost
@bors bors merged commit d8544ce into rust-lang:master Feb 12, 2020
@ljedrz ljedrz deleted the unused_stuff branch February 12, 2020 20:47
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.

8 participants