Optimize the processing of test options.

tests/co19/test_config.dart

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #library("co19_test_config");
 
 #import("../../tools/testing/dart/test_runner.dart");
 #import("../../tools/testing/dart/status_file_parser.dart");
 
 class Co19TestSuite {
@@ skipping 90 matching lines @@
                             configuration["timeout"],
                             completeHandler,
                             expectations,
                             isNegative));
       }
     }
   }
 
 
   Map testOptions(String filename) {
     RegExp testOptionsRegExp = const RegExp(@"// VMOptions=(.*)");

[Bill Hesse, 2011/11/16 13:47:24]

.* does not match newline, so these can match single lines in a multiline
string.  Add comment?

[Mads Ager (google), 2011/11/16 14:40:37]

Yes, thanks. I actually didn't think about that. I was just lucky that it
worked. ;-)

     RegExp dartOptionsRegExp = const RegExp(@"// DartOptions=(.*)");
+
+    // Read the entire file into a byte buffer and transform it to a
+    // String. This will treat the file as ascii but the only parts
+    // we are interested in will be ascii in any case.
     File file = new File(filename);
-    FileInputStream fileStream = file.openInputStream();
-    StringInputStream lines = new StringInputStream(fileStream);
+    file.openSync();
+    List chars = new List(file.lengthSync());
+    var offset = 0;
+    while (offset != chars.length) {
+      offset += file.readListSync(chars, offset, chars.length - offset);
+    }
+    file.closeSync();
+    String contents = new String.fromCharCodes(chars);
+    chars = null;
 
+    // Find the options in the file.
     List<List> result = new List<List>();
     List<String> dartOptions;
     bool isNegative = false;
-    String line;
-    while ((line = lines.readLine()) != null) {
-      Match match = testOptionsRegExp.firstMatch(line);
-      if (match != null) {
-        result.add(match[1].split(' ').filter((e) => e != ''));
+
+    Iterable<Match> matches = testOptionsRegExp.allMatches(contents);
+    for (var match in matches) {
+      result.add(match[1].split(' ').filter((e) => e != ''));
+    }
+
+    matches = dartOptionsRegExp.allMatches(contents);
+    for (var match in matches) {
+      if (dartOptions != null) {
+        throw new Exception(
+            'More than one "// DartOptions=" line in test $filename');
       }
+      dartOptions = match[1].split(' ').filter((e) => e != '');
+    }
 
-      match = dartOptionsRegExp.firstMatch(line);
-      if (match != null) {
-        if (dartOptions != null) {
-          throw new Exception(
-              'More than one "// DartOptions=" line in test $filename');
-        }
-        dartOptions = match[1].split(' ').filter((e) => e != '');
-      }
+    if (contents.contains("@compile-error") ||
+        contents.contains("@runtime-error")) {
+      isNegative = true;
+    } else if (contents.contains("@dynamic-type-error") &&
+               configuration['checked']) {
+      isNegative = true;
+    }
 
-      if (line.contains("@compile-error") || line.contains("@runtime-error")) {
-        isNegative = true;
-      } else if (line.contains("@dynamic-type-error") &&
-                 configuration['checked']) {
-        isNegative = true;
-      }
-    }
     return { "vmOptions": result,
              "dartOptions": dartOptions,
              "isNegative" : isNegative };
   }
 
-  void completeHandler(TestCase testCase) {
+  void completeHandler(TestCase test) {
   }
 }