Fix #3861: Spec changes to extractors implemented in #3747 #4024
Conversation
odersky
changed the title
|
|
||
| class Person(val name: String, val children: Person *) | ||
| object Person { | ||
| def unapplySeq(p: Person) = Some((p.name, p.children)) |
There was a problem hiding this comment.
Maybe add a paragraph explaining the rules that make an unapply qualify for vararg extraction. I am not even sure what are the rules myself. The return type must be of the form Option[Seq[T]] or Option[(..., Seq[T])]?
There was a problem hiding this comment.
I can't context switch into that, sorry. Constant context switching is what kills my productivity. So either somebody else digs out the details and adds them or we leave it as is. Again, my plea stands that somebody will have to take over the pattern matching problematic.
|
No activity for 3 weeks + 1 day. We either merge or close. Since I can't see a reason why this PR could make things worse I am merging. But we have to get better moving things along. |
Fixes #3861