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

Unified Diff: pkg/barback/lib/src/transformer/wrapping_aggregate_transformer.dart

Issue 267393009: Transition barback's infrastructure to an aggregate-based model. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: code review Created 6 years, 7 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 | « pkg/barback/lib/src/transformer/transformer.dart ('k') | pkg/barback/test/asset_set_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/barback/lib/src/transformer/wrapping_aggregate_transformer.dart
diff --git a/pkg/barback/lib/src/transformer/wrapping_aggregate_transformer.dart b/pkg/barback/lib/src/transformer/wrapping_aggregate_transformer.dart
new file mode 100644
index 0000000000000000000000000000000000000000..f41d07584a3fdcd742e6bfcd89c52081ad02518d
--- /dev/null
+++ b/pkg/barback/lib/src/transformer/wrapping_aggregate_transformer.dart
@@ -0,0 +1,84 @@
+// Copyright (c) 2014, 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 barback.transformer.wrapping_aggregate_transformer;
+
+import 'dart:async';
+
+import '../asset/asset_id.dart';
+import '../utils.dart';
+import 'aggregate_transform.dart';
+import 'aggregate_transformer.dart';
+import 'declaring_aggregate_transform.dart';
+import 'declaring_aggregate_transformer.dart';
+import 'declaring_transform.dart';
+import 'declaring_transformer.dart';
+import 'lazy_aggregate_transformer.dart';
+import 'lazy_transformer.dart';
+import 'transform.dart';
+import 'transformer.dart';
+
+/// An [AggregateTransformer] that wraps a non-aggregate [Transformer].
+///
+/// Although barback internally works in terms of [AggregateTransformer]s, most
+/// transformers only work on individual primary inputs in isolation. We want to
+/// allow those transformers to implement the more user-friendly [Transformer]
+/// interface. This class makes that possible.
+class WrappingAggregateTransformer implements AggregateTransformer {
+ /// The wrapped transformer.
+ final Transformer transformer;
+
+ factory WrappingAggregateTransformer(Transformer transformer) {
+ if (transformer is LazyTransformer) {
+ return new _LazyWrappingAggregateTransformer(
+ transformer as LazyTransformer);
+ } else if (transformer is DeclaringTransformer) {
+ return new _DeclaringWrappingAggregateTransformer(
+ transformer as DeclaringTransformer);
+ } else {
+ return new WrappingAggregateTransformer._(transformer);
+ }
+ }
+
+ WrappingAggregateTransformer._(this.transformer);
+
+ Future<String> classifyPrimary(AssetId id) {
+ return syncFuture(() => transformer.isPrimary(id))
+ .then((isPrimary) => isPrimary ? id.path : null);
+ }
+
+ Future apply(AggregateTransform aggregateTransform) {
+ return newTransform(aggregateTransform)
+ .then((transform) => transformer.apply(transform));
+ }
+
+ String toString() => transformer.toString();
+}
+
+/// A wrapper for [DeclaringTransformer]s that implements
+/// [DeclaringAggregateTransformer].
+class _DeclaringWrappingAggregateTransformer
+ extends WrappingAggregateTransformer
+ implements DeclaringAggregateTransformer {
+ final DeclaringTransformer _declaring;
+
+ _DeclaringWrappingAggregateTransformer(DeclaringTransformer transformer)
+ : _declaring = transformer,
+ super._(transformer as Transformer);
+
+ Future declareOutputs(DeclaringAggregateTransform aggregateTransform) {
+ return newDeclaringTransform(aggregateTransform).then((transform) {
+ return (transformer as DeclaringTransformer).declareOutputs(transform);
+ });
+ }
+}
+
+/// A wrapper for [LazyTransformer]s that implements
+/// [LazyAggregateTransformer].
+class _LazyWrappingAggregateTransformer
+ extends _DeclaringWrappingAggregateTransformer
+ implements LazyAggregateTransformer {
+ _LazyWrappingAggregateTransformer(LazyTransformer transformer)
+ : super(transformer);
+}
« no previous file with comments | « pkg/barback/lib/src/transformer/transformer.dart ('k') | pkg/barback/test/asset_set_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698