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

Unified Diff: pkg/barback/lib/src/package_graph.dart

Issue 107303004: Don't limit transformer parallelism in barback. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years 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/barback/lib/src/pool.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/barback/lib/src/package_graph.dart
diff --git a/pkg/barback/lib/src/package_graph.dart b/pkg/barback/lib/src/package_graph.dart
index 2ecf6b4fc9793a16eeda7c590f9067445ca4a03c..c2e1a8ee1e165f5314e82a90e2c764885219cbc9 100644
--- a/pkg/barback/lib/src/package_graph.dart
+++ b/pkg/barback/lib/src/package_graph.dart
@@ -69,13 +69,6 @@ class PackageGraph {
/// The stack trace for [_lastUnexpectedError].
StackTrace _lastUnexpectedErrorTrace;
- // TODO(nweiz): Allow transformers to declare themselves as "lightweight" or
- // "heavyweight" and adjust their restrictions appropriately. Simple
- // transformers may be very efficient to run in parallel, whereas dart2js uses
- // a lot of memory and should be run more sequentially.
- /// A pool that controls how many transformers may be applied at once.
- final Pool transformPool = new Pool(10);
-
/// Creates a new [PackageGraph] that will transform assets in all packages
/// made available by [provider].
PackageGraph(this.provider) {
« no previous file with comments | « no previous file | pkg/barback/lib/src/pool.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698