Fix #44 Accept fifth edition names

* jvm/src/main/scala/scala/xml/parsing/TokenTests.scala (isNameChar):
Add middle dot, #xB7, as specified at [4a] of XML 1.0 5th edition.

* jvm/src/test/scala/scala/xml/XMLTest.scala (t9060): New test.

* jvm/src/test/scala/scala/xml/parsing/ConstructingParserTest.scala
(t9060): New test.

Author: som-snytt

jvm/src/test/scala/scala/xml/XMLTest.scala

@@ -414,7 +414,7 @@ class XMLTest {
   }
 
   @UnitTest
-  def XmlEy {
+  def XmlEy = {
     val z = ax \ "@{the namespace from outer space}foo"
     assertTrue((ax \ "@{the namespace from outer space}foo") xml_== "baz")
     assertTrue((cx \ "@{the namespace from outer space}foo") xml_== "baz")
@@ -721,7 +721,7 @@ expected closing tag of foo
   }
 
   @UnitTest
-  def t5052 {
+  def t5052 = {
     assertTrue(<elem attr={ null: String }/> xml_== <elem/>)
     assertTrue(<elem attr={ None }/> xml_== <elem/>)
     assertTrue(<elem/> xml_== <elem attr={ null: String }/>)
@@ -743,7 +743,7 @@ expected closing tag of foo
   }
 
   @UnitTest
-  def t5843 {
+  def t5843 = {
     val foo = scala.xml.Attribute(null, "foo", "1", scala.xml.Null)
     val bar = scala.xml.Attribute(null, "bar", "2", foo)
     val ns = scala.xml.NamespaceBinding(null, "uri", scala.xml.TopScope)
@@ -775,7 +775,7 @@ expected closing tag of foo
   }
 
   @UnitTest
-  def t7074 {
+  def t7074 = {
     assertEquals("""<a/>""", sort(<a/>) toString)
     assertEquals("""<a b="2" c="3" d="1"/>""", sort(<a d="1" b="2" c="3"/>) toString)
     assertEquals("""<a b="2" c="4" d="1" e="3" f="5"/>""", sort(<a d="1" b="2" e="3" c="4" f="5"/>) toString)
@@ -787,6 +787,13 @@ expected closing tag of foo
     assertEquals("""<a a:b="1" a:c="2" a:d="3" a:e="4" a:f="5"/>""", sort(<a a:b="1" a:c="2" a:d="3" a:e="4" a:f="5"/>) toString)
   }
 
+  @UnitTest
+  def t9060 = {
+    val expected = """<a xmlns:b·="http://example.com"/>"""
+    val source = Source.fromString(expected)
+    assertEquals(expected, XML.loadString(expected).toString)
+  }
+
   @UnitTest
   def attributes = {
     val noAttr = <t/>

jvm/src/test/scala/scala/xml/parsing/ConstructingParserTest.scala

@@ -0,0 +1,26 @@
+package scala.xml
+package parsing
+
+import scala.io.Source
+import org.junit.Test
+import org.junit.Ignore
+import org.junit.runner.RunWith
+import org.junit.runners.JUnit4
+import org.junit.Assert
+import Assert._
+import scala.xml.JUnitAssertsForXML.{ assertEquals => assertXml }
+
+class ConstructingParserTest {
+
+  @Test
+  def t9060 = {
+    val a = "<a xmlns:b·='http://example.com'/>"
+    val source = new Source {
+      val iter = a.iterator
+      override def reportError(pos: Int, msg: String, out: java.io.PrintStream = Console.err) {}
+    }
+    val doc = ConstructingParser.fromSource(source, false).content(TopScope)
+
+  }
+
+}

shared/src/main/scala/scala/xml/parsing/TokenTests.scala

@@ -37,10 +37,10 @@ trait TokenTests {
 
   /**
    * {{{
-   *  NameChar ::= Letter | Digit | '.' | '-' | '_' | ':'
+   *  NameChar ::= Letter | Digit | '.' | '-' | '_' | ':' | #xB7
    *             | CombiningChar | Extender
    *  }}}
-   *  See [4] and Appendix B of XML 1.0 specification.
+   *  See [4] and [4a] of Appendix B of XML 1.0 specification.
    */
   def isNameChar(ch: Char) = {
     import java.lang.Character._
@@ -50,7 +50,7 @@ trait TokenTests {
       case COMBINING_SPACING_MARK |
         ENCLOSING_MARK | NON_SPACING_MARK |
         MODIFIER_LETTER | DECIMAL_DIGIT_NUMBER => true
-      case _ => ".-:" contains ch
+      case _ => ".-:·" contains ch
     })
   }