-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Assertion failure when committing TyperState #12736
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
Comments
Got a similar assertion failure while renaming and refactoring files. But I am unsure where this came from or it is the same issue. Details
|
It looks like the same issue yes. |
smarter
added a commit
to dotty-staging/dotty
that referenced
this issue
Jun 15, 2021
Flushing the reporter can force messages (apparently because of `HideNonSensicalMessages#isHidden`) which can affect the TyperState we're in the process of committing, so make sure that flushing happens before committing to not trigger any assertion. Fixes scala#12736
smarter
added a commit
to dotty-staging/dotty
that referenced
this issue
Jun 15, 2021
Flushing the reporter can force messages (apparently because of `HideNonSensicalMessages#isHidden`) which can affect the TyperState we're in the process of committing, so make sure that flushing happens before committing to not trigger any assertion. Fixes scala#12736
EnzeXing
pushed a commit
to EnzeXing/dotty
that referenced
this issue
Jun 15, 2021
Flushing the reporter can force messages (apparently because of `HideNonSensicalMessages#isHidden`) which can affect the TyperState we're in the process of committing, so make sure that flushing happens before committing to not trigger any assertion. Fixes scala#12736
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Compiler version
3.0.1-RC1 from sbt
Minimized code
Output (click arrow to expand)
FWIW it came up while adding an additional type parameter to a trait --- but oddly enough, I was able to prune that trait away while minimizing this example.
The text was updated successfully, but these errors were encountered: