Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(126)

Unified Diff: pkg/analyzer/bin/analyzer.dart

Issue 135593009: Fixes for exit problems. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | pkg/analyzer/lib/src/services/runtime/coverage/coverage_impl.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analyzer/bin/analyzer.dart
diff --git a/pkg/analyzer/bin/analyzer.dart b/pkg/analyzer/bin/analyzer.dart
index 76510c0b8cf772e95af781be8e545c64d1974851..19de8caa77e50b392c46c5b0930cff3d3c469b24 100644
--- a/pkg/analyzer/bin/analyzer.dart
+++ b/pkg/analyzer/bin/analyzer.dart
@@ -95,7 +95,7 @@ class BatchRunner {
* Run the tool in 'batch' mode, receiving command lines through stdin and returning pass/fail
* status through stdout. This feature is intended for use in unit testing.
*/
- static ErrorSeverity runAsBatch(List<String> sharedArgs, BatchRunnerHandler handler) {
+ static void runAsBatch(List<String> sharedArgs, BatchRunnerHandler handler) {
stdout.writeln('>>> BATCH START');
Stopwatch stopwatch = new Stopwatch();
stopwatch.start();
@@ -111,7 +111,7 @@ class BatchRunner {
if (line.isEmpty) {
var time = stopwatch.elapsedMilliseconds;
stdout.writeln('>>> BATCH END (${totalTests - testsFailed}/$totalTests) ${time}ms');
- exit(batchResult.ordinal);
+ exitCode = batchResult.ordinal;
}
// prepare aruments
var args;
« no previous file with comments | « no previous file | pkg/analyzer/lib/src/services/runtime/coverage/coverage_impl.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698