about summary refs log tree commit diff
diff options
context:
space:
mode:
-rw-r--r--doc/tutorial.md9
-rw-r--r--src/test/compile-fail/bad-for-loop-2.rs15
-rw-r--r--src/test/compile-fail/bad-for-loop.rs7
3 files changed, 24 insertions, 7 deletions
diff --git a/doc/tutorial.md b/doc/tutorial.md
index 14a0c6fdcfe..8caf8704e05 100644
--- a/doc/tutorial.md
+++ b/doc/tutorial.md
@@ -1613,18 +1613,19 @@ loop. Like `do`, `for` is a nice syntax for describing control flow
 with closures.  Additionally, within a `for` loop, `break`, `loop`,
 and `return` work just as they do with `while` and `loop`.
 
-Consider again our `each` function, this time improved to
-break early when the iteratee returns `false`:
+Consider again our `each` function, this time improved to return
+immediately when the iteratee returns `false`:
 
 ~~~~
-fn each(v: &[int], op: &fn(v: &int) -> bool) {
+fn each(v: &[int], op: &fn(v: &int) -> bool) -> bool {
    let mut n = 0;
    while n < v.len() {
        if !op(&v[n]) {
-           break;
+           return false;
        }
        n += 1;
    }
+   return true;
 }
 ~~~~
 
diff --git a/src/test/compile-fail/bad-for-loop-2.rs b/src/test/compile-fail/bad-for-loop-2.rs
new file mode 100644
index 00000000000..4f90ff5802d
--- /dev/null
+++ b/src/test/compile-fail/bad-for-loop-2.rs
@@ -0,0 +1,15 @@
+// Copyright 2012 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 main() {
+    fn quux(_: &fn(&int) -> bool) -> () { }
+    for quux |_| { } //~ ERROR expected `for` closure to return
+                     // `bool`, but found `()`
+}
diff --git a/src/test/compile-fail/bad-for-loop.rs b/src/test/compile-fail/bad-for-loop.rs
index 7ff51eff8ee..83616b708f2 100644
--- a/src/test/compile-fail/bad-for-loop.rs
+++ b/src/test/compile-fail/bad-for-loop.rs
@@ -9,7 +9,8 @@
 // except according to those terms.
 
 fn main() {
-    fn baz(_x: &fn(y: int) -> int) {}
-    for baz |_e| { } //~ ERROR A `for` loop iterator should expect a closure that returns `bool`
-                     //~^ ERROR expected `for` closure to return `bool`
+    fn quux(_: &fn(&int) -> int) -> bool { true }
+    for quux |_| { } //~ ERROR A `for` loop iterator should expect a
+                     // closure that returns `bool`.  This iterator
+                     // expects a closure that returns `int`.
 }