Skip to content

Add While and DoWhile extractors Tasty reflect #4630

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

Merged
merged 1 commit into from
Jun 22, 2018

Conversation

nicolasstucki
Copy link
Contributor

No description provided.

@nicolasstucki
Copy link
Contributor Author

Rebased

@nicolasstucki nicolasstucki force-pushed the add-while-to-tasty-relect branch from 948c6ea to 0812693 Compare June 21, 2018 14:04
@nicolasstucki
Copy link
Contributor Author

Rebased

@nicolasstucki nicolasstucki force-pushed the add-while-to-tasty-relect branch from 0812693 to 599ee65 Compare June 21, 2018 15:42
@nicolasstucki
Copy link
Contributor Author

Rebased

case _ => None
}
private def normalizedLoops(tree: tpd.Tree)(implicit ctx: Context): tpd.Tree = tree match {
case block: tpd.Block =>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could be case block: tpd.Block if block.stats.size > 1 =>

case Trees.Block(stats, expr) => Some((stats, expr))
case _ => None
}
private def normalizedLoops(tree: tpd.Tree)(implicit ctx: Context): tpd.Tree = tree match {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs a comment. It is not clear at all what it does

@nicolasstucki nicolasstucki force-pushed the add-while-to-tasty-relect branch from 599ee65 to d143026 Compare June 22, 2018 11:41
@nicolasstucki nicolasstucki merged commit 6fb7970 into scala:master Jun 22, 2018
@allanrenucci allanrenucci deleted the add-while-to-tasty-relect branch June 22, 2018 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants