---

yaml
---
r: 15748
b: refs/heads/try
c: 37f2330
h: refs/heads/master
v: v3

Author: catamorphism

[refs]

@@ -2,5 +2,5 @@
 refs/heads/master: 61b1875c16de39c166b0f4d54bba19f9c6777d1a
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 4a81779abd786ff22d71434c6d9a5917ea4cdfff
-refs/heads/try: 82d4fe3967a6e6b7bde5d70644cf3eca23763932
+refs/heads/try: 37f23303e63b16c1b9b2726d8dc464c44580abb1
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105

branches/try/src/compiletest/header.rs

@@ -29,7 +29,7 @@ fn load_props(testfile: str) -> test_props {
     let mut exec_env = [];
     let mut compile_flags = option::none;
     let mut pp_exact = option::none;
-    iter_header(testfile) {|ln|
+    for iter_header(testfile) {|ln|
         alt parse_error_pattern(ln) {
           option::some(ep) { error_patterns += [ep]; }
           option::none { }
@@ -62,14 +62,11 @@ fn load_props(testfile: str) -> test_props {
 
 fn is_test_ignored(config: config, testfile: str) -> bool {
     let mut found = false;
-    iter_header(testfile) {|ln|
-        // FIXME: Can't return or break from iterator
-        // (Fix when Issue #1619 is resolved)
-        found = found || parse_name_directive(ln, "xfail-test");
-        found = found || parse_name_directive(ln, xfail_target());
-        if (config.mode == common::mode_pretty) {
-            found = found || parse_name_directive(ln, "xfail-pretty");
-        }
+    for iter_header(testfile) {|ln|
+        if parse_name_directive(ln, "xfail-test") { ret true; }
+        if parse_name_directive(ln, xfail_target()) { ret true; }
+        if config.mode == common::mode_pretty &&
+           parse_name_directive(ln, "xfail-pretty") { ret true; }
     };
     ret found;
 
@@ -78,7 +75,7 @@ fn is_test_ignored(config: config, testfile: str) -> bool {
     }
 }
 
-fn iter_header(testfile: str, it: fn(str)) {
+fn iter_header(testfile: str, it: fn(str) -> bool) -> bool {
     let rdr = result::get(io::file_reader(testfile));
     while !rdr.eof() {
         let ln = rdr.read_line();
@@ -88,9 +85,10 @@ fn iter_header(testfile: str, it: fn(str)) {
         // with a warm page cache. Maybe with a cold one.
         if str::starts_with(ln, "fn")
             || str::starts_with(ln, "mod") {
-            break;
-        } else { it(ln); }
+            ret false;
+        } else { if !(it(ln)) { ret false; } }
     }
+    ret true;
 }
 
 fn parse_error_pattern(line: str) -> option<str> {