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

Revert "Make LocalAnalizer visitor iterate instead of recurse" #61641

Merged

Conversation

Projects
None yet
4 participants
@spastorino
Copy link
Member

commented Jun 7, 2019

This reverts commit 0cfaa28.

r? @oli-obk

@oli-obk

This comment has been minimized.

Copy link
Contributor

commented Jun 7, 2019

@bors r+

@bors

This comment has been minimized.

Copy link
Contributor

commented Jun 7, 2019

📌 Commit 4c32617 has been approved by oli-obk

Centril added a commit to Centril/rust that referenced this pull request Jun 7, 2019

Rollup merge of rust-lang#61641 - spastorino:revert-to-recursion-on-l…
…ocal-analyzer, r=oli-obk

Revert "Make LocalAnalizer visitor iterate instead of recurse"

This reverts commit 0cfaa28.

r? @oli-obk

@Centril Centril referenced this pull request Jun 7, 2019

Closed

Rollup of 7 pull requests #61644

bors added a commit that referenced this pull request Jun 8, 2019

Auto merge of #61644 - Centril:rollup-tjx5kx9, r=Centril
Rollup of 7 pull requests

Successful merges:

 - #61223 (Document tuple's Ord behavior as sequential)
 - #61597 (Bump dirs, rand and redox_users)
 - #61615 (syntax: Treat error literals in more principled way)
 - #61616 (parser: Remove `Deref` impl from `Parser`)
 - #61621 (Clarify when we run steps with ONLY_HOSTS)
 - #61627 (Add regression test for #61452.)
 - #61641 (Revert "Make LocalAnalizer visitor iterate instead of recurse")

Failed merges:

r? @ghost

Centril added a commit to Centril/rust that referenced this pull request Jun 8, 2019

Rollup merge of rust-lang#61641 - spastorino:revert-to-recursion-on-l…
…ocal-analyzer, r=oli-obk

Revert "Make LocalAnalizer visitor iterate instead of recurse"

This reverts commit 0cfaa28.

r? @oli-obk

@Centril Centril referenced this pull request Jun 8, 2019

Merged

Rollup of 7 pull requests #61649

bors added a commit that referenced this pull request Jun 8, 2019

Auto merge of #61649 - Centril:rollup-b4nx9k9, r=Centril
Rollup of 7 pull requests

Successful merges:

 - #61223 (Document tuple's Ord behavior as sequential)
 - #61615 (syntax: Treat error literals in more principled way)
 - #61616 (parser: Remove `Deref` impl from `Parser`)
 - #61621 (Clarify when we run steps with ONLY_HOSTS)
 - #61627 (Add regression test for #61452.)
 - #61641 (Revert "Make LocalAnalizer visitor iterate instead of recurse")
 - #61647 (Use stable wrappers in f32/f64::signum)

Failed merges:

r? @ghost

@bors bors merged commit 4c32617 into rust-lang:master Jun 8, 2019

1 check passed

Travis CI - Pull Request Build Passed
Details
@bors

This comment has been minimized.

Copy link
Contributor

commented Jun 8, 2019

⌛️ Testing commit 4c32617 with merge 6312b89...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.