about summary refs log tree commit diff
diff options
context:
space:
mode:
authorManuel Hoffmann <manuel@polythematik.de>2015-04-13 15:56:10 +0200
committerManuel Hoffmann <manuel@polythematik.de>2015-04-13 15:56:10 +0200
commit4abade50d7096ee5585255ca08d69b1ccd73d09f (patch)
treee677c8d3cbe7e588088410b08f53bb2a55666bfb
parent0a2885ad944aa1a5f60a72a7551b1b45367637f6 (diff)
Added a help span which informs the user about the escaping of curly braces in a format string if a wrongly escaped one is detected in a string.
-rw-r--r--src/libsyntax/parse/lexer/mod.rs8
-rw-r--r--src/test/parse-fail/wrong-escape-of-curly-braces.rs18
2 files changed, 25 insertions, 1 deletions
diff --git a/src/libsyntax/parse/lexer/mod.rs b/src/libsyntax/parse/lexer/mod.rs
index 22b7d5c9f1d..f891318659a 100644
--- a/src/libsyntax/parse/lexer/mod.rs
+++ b/src/libsyntax/parse/lexer/mod.rs
@@ -843,13 +843,19 @@ impl<'a> StringReader<'a> {
                                     if ascii_only { "unknown byte escape" }
                                     else { "unknown character escape" },
                                     c);
+                                let sp = codemap::mk_sp(escaped_pos, last_pos);
                                 if e == '\r' {
-                                    let sp = codemap::mk_sp(escaped_pos, last_pos);
                                     self.span_diagnostic.span_help(
                                         sp,
                                         "this is an isolated carriage return; consider checking \
                                          your editor and version control settings")
                                 }
+                                if (e == '{' || e == '}') && !ascii_only {
+                                    self.span_diagnostic.span_help(
+                                        sp,
+                                        "if used in a formatting string, \
+                                        curly braces are escaped with `{{` and `}}`")
+                                }
                                 false
                             }
                         }
diff --git a/src/test/parse-fail/wrong-escape-of-curly-braces.rs b/src/test/parse-fail/wrong-escape-of-curly-braces.rs
new file mode 100644
index 00000000000..d86bf726cbd
--- /dev/null
+++ b/src/test/parse-fail/wrong-escape-of-curly-braces.rs
@@ -0,0 +1,18 @@
+// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
+// file at the top-level directory of this distribution and at
+// http://rust-lang.org/COPYRIGHT.
+//
+// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
+// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
+// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
+// option. This file may not be copied, modified, or distributed
+// except according to those terms.
+
+fn f() {
+    let ok = "{{everything fine}}";
+    let bad = "\{it is wrong\}";
+    //~^  ERROR unknown character escape: {
+    //~^^  HELP if used in a formatting string, curly braces are escaped with `{{` and `}}`
+    //~^^^ ERROR unknown character escape: }
+    //~^^^^  HELP if used in a formatting string, curly braces are escaped with `{{` and `}}`
+}