Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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
Contributor

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
Collaborator

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