Name-based pattern matching dropped for unapplySeq in 2.13

Author: SethTisue

Diff for: shared/src/main/scala/scala/xml/Elem.scala

@@ -36,7 +36,7 @@ object Elem {
 
   def unapplySeq(n: Node) = n match {
     case _: SpecialNode | _: Group => None
-    case _                         => Some((n.prefix, n.label, n.attributes, n.scope, n.child))
+    case _                         => Some((n.prefix, n.label, n.attributes, n.scope, n.child.toSeq))
   }
 
   import scala.sys.process._

Diff for: shared/src/main/scala/scala/xml/Node.scala

@@ -24,7 +24,7 @@ object Node {
   /** the empty namespace */
   val EmptyNamespace = ""
 
-  def unapplySeq(n: Node) = Some((n.label, n.attributes, n.child))
+  def unapplySeq(n: Node) = Some((n.label, n.attributes, n.child.toSeq))
 }
 
 /**

Diff for: shared/src/main/scala/scala/xml/QNode.scala

@@ -16,5 +16,5 @@ package xml
  *  @author  Burak Emir
  */
 object QNode {
-  def unapplySeq(n: Node) = Some((n.scope.getURI(n.prefix), n.label, n.attributes, n.child))
+  def unapplySeq(n: Node) = Some((n.scope.getURI(n.prefix), n.label, n.attributes, n.child.toSeq))
 }

Diff for: shared/src/test/scala/scala/xml/PatternMatching.scala

@@ -57,10 +57,10 @@ class PatternMatching extends {
   }
 
   object SafeNodeSeq {
-    def unapplySeq(any: Any): Option[Seq[Node]] = any match {
-      case s: Seq[_] => Some(s flatMap (_ match {
+    def unapplySeq(any: Any): Option[collection.immutable.Seq[Node]] = any match {
+      case s: Seq[_] => Some((s flatMap (_ match {
         case n: Node => n case _ => NodeSeq.Empty
-      })) case _ => None
+      })).toSeq) case _ => None
     }
   }