summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSom Snytt <som.snytt@gmail.com>2014-12-16 13:43:17 -0800
committerSom Snytt <som.snytt@gmail.com>2014-12-16 13:43:17 -0800
commit4d96a8ad0b7050303e0b841708ba452e708f6a35 (patch)
treecde02ccc18020efece1ab5e76616b520736572c9
parent7ba38a07916426314cc3bff6999f3992757e0b26 (diff)
downloadscala-4d96a8ad0b7050303e0b841708ba452e708f6a35.tar.gz
scala-4d96a8ad0b7050303e0b841708ba452e708f6a35.tar.bz2
scala-4d96a8ad0b7050303e0b841708ba452e708f6a35.zip
SI-5154 Parse leading literal brace in XML pattern
Don't consume literal brace as Scala pattern. Previously, leading space would let the text parser `xText` handle it correctly instead.
-rwxr-xr-xsrc/compiler/scala/tools/nsc/ast/parser/MarkupParsers.scala7
-rw-r--r--test/files/pos/t5154.scala9
2 files changed, 12 insertions, 4 deletions
diff --git a/src/compiler/scala/tools/nsc/ast/parser/MarkupParsers.scala b/src/compiler/scala/tools/nsc/ast/parser/MarkupParsers.scala
index f1517e56a0..96939e616c 100755
--- a/src/compiler/scala/tools/nsc/ast/parser/MarkupParsers.scala
+++ b/src/compiler/scala/tools/nsc/ast/parser/MarkupParsers.scala
@@ -425,11 +425,10 @@ trait MarkupParsers {
if (ch != '/') ts append xPattern // child
else return false // terminate
- case '{' => // embedded Scala patterns
- while (ch == '{') {
- nextch()
+ case '{' if xCheckEmbeddedBlock => // embedded Scala patterns, if not double brace
+ do {
ts ++= xScalaPatterns
- }
+ } while (xCheckEmbeddedBlock)
assert(!xEmbeddedBlock, "problem with embedded block")
case SU =>
diff --git a/test/files/pos/t5154.scala b/test/files/pos/t5154.scala
new file mode 100644
index 0000000000..2629308f00
--- /dev/null
+++ b/test/files/pos/t5154.scala
@@ -0,0 +1,9 @@
+
+trait Z {
+ // extra space made the pattern OK
+ def f = <z> {{3}}</z> match { case <z> {{3}}</z> => }
+
+ // lack of space: error: illegal start of simple pattern
+ def g = <z>{{3}}</z> match { case <z>{{3}}</z> => }
+}
+