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

Unified Diff: sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart

Issue 598883002: Fix an error-swallowing bug in pub and several bugs it was hiding. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 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: sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart
diff --git a/sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart b/sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart
index 4ac4dae1531538b7059cb321ab7b11b97c1c9d98..4f760357a362d832588b90907601aa10e4f3c4d8 100644
--- a/sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart
+++ b/sdk/lib/_internal/pub_generated/lib/src/barback/load_all_transformers.dart
@@ -47,9 +47,16 @@ Future loadAllTransformers(AssetEnvironment environment,
var transformers =
environment.getBuiltInTransformers(package);
join0() {
- newFuture(
- (() => environment.barback.updateTransformers(package.name, phases)));
- completer0.complete(null);
+ join1() {
+ newFuture(
+ (() => environment.barback.updateTransformers(package.name, phases)));
+ completer0.complete(null);
+ }
+ if (phases.isEmpty) {
+ completer0.complete(null);
+ } else {
+ join1();
+ }
}
if (transformers != null) {
phases.add(transformers);

Powered by Google App Engine
This is Rietveld 408576698