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

Better error reporting for patterns #8317

Merged
merged 4 commits into from Feb 17, 2020

Conversation

@odersky
Copy link
Contributor

odersky commented Feb 14, 2020

Previously, failed pattern match typings gave too often the message:

D cannot be used as an extractor in a pattern because it lacks an unapply or unapplySeq method

This message sometimes hid some error condition that was more informative. We now report these errors instead.

odersky added 4 commits Feb 14, 2020
Contained is a pure function, so the () was removed.
@anatoliykmetyuk anatoliykmetyuk merged commit 516978a into lampepfl:master Feb 17, 2020
4 checks passed
4 checks passed
test
Details
test_bootstrapped
Details
CLA User signed CLA
Details
continuous-integration/drone/pr Build is passing
Details
@anatoliykmetyuk anatoliykmetyuk deleted the dotty-staging:fix-unapply-reporting branch Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.