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

regression in xml lib #5858

Closed
scabug opened this issue May 31, 2012 · 4 comments
Closed

regression in xml lib #5858

scabug opened this issue May 31, 2012 · 4 comments
Assignees
Milestone

Comments

@scabug
Copy link

scabug commented May 31, 2012

Worked in 2.9.


scala> new xml.Elem(null, null, xml.Null, xml.TopScope, Nil: _*)
<console>:8: error: ambiguous reference to overloaded definition,
both constructor Elem in class Elem of type (prefix: String, label: String, attributes: scala.xml.MetaData, scope: scala.xml.NamespaceBinding, child: scala.xml.Node*)scala.xml.Elem
and  constructor Elem in class Elem of type (prefix: String, label: String, attributes1: scala.xml.MetaData, scope: scala.xml.NamespaceBinding, minimizeEmpty: Boolean, child: scala.xml.Node*)scala.xml.Elem
match argument types (Null,Null,xml.Null.type,xml.TopScope.type,Nothing)
              new xml.Elem(null, null, xml.Null, xml.TopScope, Nil: _*)
              ^
@scabug
Copy link
Author

scabug commented May 31, 2012

Imported From: https://issues.scala-lang.org/browse/SI-5858?orig=1
Reporter: @paulp
Affected Versions: 2.10.0
See #5859

@scabug
Copy link
Author

scabug commented Jun 1, 2012

@som-snytt said:
See #1118
but this works

xmlover.scala:10: warning: constructor Elem in class Elem is deprecated: This constructor is retained for backward compatibility. Please use the primary constructor, which lets you specify your own preference for `minimizeEmpty`.
    val y = new xml.Elem(null, null, xml.Null, xml.TopScope, (Nil: Seq[Node]): _*)

"But that's a Boolean param!"

I really am rushing off to pick up at pre-K, that's not just an exit line.

@scabug
Copy link
Author

scabug commented Dec 8, 2012

@retronym said:
Fixed by #5859; closing with a test case: scala/scala#1731

@scabug
Copy link
Author

scabug commented Jan 22, 2013

@adriaanm said:
reopening for 2.10.1-RC1 backport

@scabug scabug closed this as completed Feb 4, 2013
@scabug scabug added this to the 2.10.1 milestone Apr 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants