Skip to content

Conversation

@DarkDimius
Copy link
Contributor

No description provided.

@smarter
Copy link
Member

smarter commented Mar 25, 2015

This correctly fixes #431 and an issue I was having with value classes and pattern matching, I'm not familiar with PatternMatcher.scala so I can't say anything about the code itself.

@odersky
Copy link
Contributor

odersky commented Mar 26, 2015

Neither can I. Formally it looks OK to me. So I leave it to @DarkDimius to merge if he thinks we should.

@DarkDimius
Copy link
Contributor Author

Let's merge it as @smarter relies on it.

DarkDimius added a commit that referenced this pull request Mar 30, 2015
Fix #431: Options make 2 steps of decoding instead of one
@DarkDimius DarkDimius merged commit 2362081 into scala:master Mar 30, 2015
odersky added a commit to dotty-staging/dotty that referenced this pull request Apr 17, 2015
Idents of lifted symbols become class members, need to carry the right
reference with the right prefix as type.
@odersky odersky mentioned this pull request Apr 17, 2015
@allanrenucci allanrenucci deleted the patmat-options branch December 14, 2017 19:24
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Jul 3, 2025
Backport "chore: have a better error message when context bounds are not allowed" to 3.3 LTS
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.

3 participants