diff options
author | Som Snytt <som.snytt@gmail.com> | 2014-12-16 13:43:17 -0800 |
---|---|---|
committer | Som Snytt <som.snytt@gmail.com> | 2014-12-16 13:43:17 -0800 |
commit | 4d96a8ad0b7050303e0b841708ba452e708f6a35 (patch) | |
tree | cde02ccc18020efece1ab5e76616b520736572c9 | |
parent | 7ba38a07916426314cc3bff6999f3992757e0b26 (diff) | |
download | scala-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-x | src/compiler/scala/tools/nsc/ast/parser/MarkupParsers.scala | 7 | ||||
-rw-r--r-- | test/files/pos/t5154.scala | 9 |
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> => } +} + |