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

Unified Diff: pkg/analyzer/lib/src/task/driver.dart

Issue 1327363002: Fixed the last unit test that was failing with the new task model (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 3 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
Index: pkg/analyzer/lib/src/task/driver.dart
diff --git a/pkg/analyzer/lib/src/task/driver.dart b/pkg/analyzer/lib/src/task/driver.dart
index e4210d50ec5e88f9c0f2855d6ae038ba5b420627..8daa5cfe08d0e1839aa5be0742a9c48e81ed5f89 100644
--- a/pkg/analyzer/lib/src/task/driver.dart
+++ b/pkg/analyzer/lib/src/task/driver.dart
@@ -102,7 +102,12 @@ class AnalysisDriver {
WorkOrder workOrder = createWorkOrderForResult(target, result);
if (workOrder != null) {
while (workOrder.moveNext()) {
+// AnalysisTask previousTask = task;
+// String message = workOrder.current.toString();
scheglov 2015/09/10 20:35:38 Do you still need this?
Brian Wilkerson 2015/09/10 21:00:31 Technically, no, but it was *very* useful for debu
task = performWorkItem(workOrder.current);
+// if (task == null) {
+// throw new AnalysisException(message, previousTask.caughtException);
+// }
}
}
return task;

Powered by Google App Engine
This is Rietveld 408576698