diff options
| author | Manuel Hoffmann <manuel@polythematik.de> | 2015-04-13 15:56:10 +0200 |
|---|---|---|
| committer | Manuel Hoffmann <manuel@polythematik.de> | 2015-04-13 15:56:10 +0200 |
| commit | 4abade50d7096ee5585255ca08d69b1ccd73d09f (patch) | |
| tree | e677c8d3cbe7e588088410b08f53bb2a55666bfb | |
| parent | 0a2885ad944aa1a5f60a72a7551b1b45367637f6 (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.rs | 8 | ||||
| -rw-r--r-- | src/test/parse-fail/wrong-escape-of-curly-braces.rs | 18 |
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 `}}` +} |
