Skip to content

Conversation

@nicolasstucki
Copy link
Contributor

No description provided.

@nicolasstucki nicolasstucki requested a review from liufengyun July 1, 2019 11:46
@nicolasstucki nicolasstucki marked this pull request as ready for review July 1, 2019 12:02
Copy link
Contributor

@liufengyun liufengyun left a comment

Choose a reason for hiding this comment

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

LGTM


override protected def picklerPhases: List[List[Phase]] =
List(List(new Staging))
List(List(new ReifyQuotes))
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious, does it still check phase consistency?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This code is there from when we had a single Staging phase that did both PCP and reification.
When I split it into Staging and ReifyQuotes I should have changed it. Unfortunately we did not have a test case that covered it.

@nicolasstucki nicolasstucki merged commit 09e7e47 into scala:master Jul 1, 2019
@liufengyun liufengyun deleted the fix-#6754 branch July 1, 2019 12:19
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