]> git.lizzy.rs Git - rust.git/commitdiff
Make lifetime elision help more useful on type signatures.
authorcrhino <piraino.chris@gmail.com>
Wed, 24 Dec 2014 07:57:35 +0000 (02:57 -0500)
committercrhino <piraino.chris@gmail.com>
Mon, 29 Dec 2014 00:26:00 +0000 (16:26 -0800)
Fixes #19707.

src/librustc_typeck/astconv.rs
src/test/compile-fail/issue-19707.rs [new file with mode: 0644]

index ff577d2d45d4bdf001e32a4a46aaed8d31645dbd..c7dcdba551b70e11f42f3a5317a49ef6224e03e3 100644 (file)
@@ -164,10 +164,16 @@ pub fn opt_ast_region_to_region<'tcx, AC: AstConv<'tcx>, RS: RegionScope>(
                             let mut m = String::new();
                             let len = v.len();
                             for (i, (name, n)) in v.into_iter().enumerate() {
-                                m.push_str(if n == 1 {
+                                let help_name = if name.is_empty() {
+                                    format!("argument {}", i + 1)
+                                } else {
                                     format!("`{}`", name)
+                                };
+
+                                m.push_str(if n == 1 {
+                                    help_name
                                 } else {
-                                    format!("one of `{}`'s {} elided lifetimes", name, n)
+                                    format!("one of {}'s {} elided lifetimes", help_name, n)
                                 }[]);
 
                                 if len == 2 && i == 0 {
diff --git a/src/test/compile-fail/issue-19707.rs b/src/test/compile-fail/issue-19707.rs
new file mode 100644 (file)
index 0000000..9affb44
--- /dev/null
@@ -0,0 +1,20 @@
+// Copyright 2014 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.
+
+#![feature(unboxed_closures)]
+#![allow(dead_code)]
+
+type foo = fn(&u8, &u8) -> &u8; //~ ERROR missing lifetime specifier
+//~^ HELP the signature does not say whether it is borrowed from argument 1 or argument 2
+
+fn bar<F: Fn(&u8, &u8) -> &u8>(f: &F) {} //~ ERROR missing lifetime specifier
+//~^ HELP the signature does not say whether it is borrowed from argument 1 or argument 2
+
+fn main() {}