diff options
| author | Davis Silverman <sinistersnare@gmail.com> | 2014-03-16 16:46:02 -0400 |
|---|---|---|
| committer | Davis Silverman <sinistersnare@gmail.com> | 2014-03-16 16:46:02 -0400 |
| commit | 8b6592ef1a29360716e49545cff42d7103366b02 (patch) | |
| tree | 04cdc113c38719d08d630aa4a4dc2632af36e17d | |
| parent | b4d324334cb48198c27d782002d75eba14a6abde (diff) | |
| download | rust-8b6592ef1a29360716e49545cff42d7103366b02.tar.gz rust-8b6592ef1a29360716e49545cff42d7103366b02.zip | |
Asked if missing (), then asks about an anonymous function. Also added test.
| -rw-r--r-- | src/librustc/middle/typeck/check/mod.rs | 6 | ||||
| -rw-r--r-- | src/test/compile-fail/method-missing-call.rs | 34 |
2 files changed, 38 insertions, 2 deletions
diff --git a/src/librustc/middle/typeck/check/mod.rs b/src/librustc/middle/typeck/check/mod.rs index 51efcb7d1c3..38a9119604b 100644 --- a/src/librustc/middle/typeck/check/mod.rs +++ b/src/librustc/middle/typeck/check/mod.rs @@ -2317,11 +2317,13 @@ fn check_expr_with_unifier(fcx: @FnCtxt, fcx.type_error_message( expr.span, |actual| { - format!("attempted to take value of method `{}` on type `{}` \ - (try writing an anonymous function)", + format!("attempted to take value of method `{}` on type `{}`", token::get_name(field), actual) }, expr_t, None); + + tcx.sess.span_note(expr.span, + "maybe a missing `()` to call it? If not, try an anonymous function."); } None => { diff --git a/src/test/compile-fail/method-missing-call.rs b/src/test/compile-fail/method-missing-call.rs new file mode 100644 index 00000000000..55a178af961 --- /dev/null +++ b/src/test/compile-fail/method-missing-call.rs @@ -0,0 +1,34 @@ +// 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. + +// Tests to make sure that parens are needed for method calls without arguments. +// outputs text to make sure either an anonymous function is provided or +// open-close '()' parens are given + + +struct Point { + x: int, + y: int +} +impl Point { + fn new() -> Point { + Point{x:0, y:0} + } + fn get_x(&self) -> int { + self.x + } +} + +fn main() { + let point: Point = Point::new(); + let px: int = point.get_x;//~ ERROR attempted to take value of method `get_x` on type `Point` + //~^ NOTE maybe a missing `()` to call it? If not, try an anonymous function. +} + |
