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

Side by Side Diff: pkg/barback/CHANGELOG.md

Issue 243103003: Run declaring transformers eagerly if possible, even if their inputs are deferred. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: code review Created 6 years, 8 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | pkg/barback/lib/src/transform_node.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 ## 0.13.1 1 ## 0.13.1
2 2
3 * Only run `Transformer.isPrimary` once for each asset. 3 * Only run `Transformer.isPrimary` once for each asset.
4 4
5 * Don't warn if a lazy or declaring transformer doesn't emit outputs that it has 5 * Don't warn if a lazy or declaring transformer doesn't emit outputs that it has
6 declared. This is valid for transformers like dart2js that need to read their 6 declared. This is valid for transformers like dart2js that need to read their
7 primary input in order to determine whether they should run. 7 primary input in order to determine whether they should run.
8 8
9 * Fix a deadlock bug when a lazy primary input to a lazy transformer became 9 * Fix a deadlock bug when a lazy primary input to a lazy transformer became
10 dirty while the transformer's `apply` method was running. 10 dirty while the transformer's `apply` method was running.
11 11
12 * Run declaring transformers with lazy inputs eagerly if the inputs become
13 available.
14
12 ## 0.13.0 15 ## 0.13.0
13 16
14 * `Transformer.isPrimary` now takes an `AssetId` rather than an `Asset`. 17 * `Transformer.isPrimary` now takes an `AssetId` rather than an `Asset`.
15 18
16 * `DeclaringTransform` now only exposes the primary input's `AssetId`, rather 19 * `DeclaringTransform` now only exposes the primary input's `AssetId`, rather
17 than the primary `Asset` object. 20 than the primary `Asset` object.
18 21
19 * `DeclaringTransform` no longer supports `getInput`, `readInput`, 22 * `DeclaringTransform` no longer supports `getInput`, `readInput`,
20 `readInputAsString`, or `hasInput`. 23 `readInputAsString`, or `hasInput`.
21 24
(...skipping 22 matching lines...) Expand all
44 47
45 * If a transformer calls `Transform.logger.error`, the transformer will now be 48 * If a transformer calls `Transform.logger.error`, the transformer will now be
46 considered to have failed after it finishes running `apply()`. This means that 49 considered to have failed after it finishes running `apply()`. This means that
47 its outputs will not be consumed by future transformers and its primary input 50 its outputs will not be consumed by future transformers and its primary input
48 will not be passed through to the next phase. 51 will not be passed through to the next phase.
49 52
50 * If a transform calls `Transform.getInput`, `Transform.readInput`, 53 * If a transform calls `Transform.getInput`, `Transform.readInput`,
51 `Transform.readInputAsString`, or `Transform.hasInput` on an input that 54 `Transform.readInputAsString`, or `Transform.hasInput` on an input that
52 doesn't exist, the transform will be re-run if that input is created in the 55 doesn't exist, the transform will be re-run if that input is created in the
53 future. 56 future.
OLDNEW
« no previous file with comments | « no previous file | pkg/barback/lib/src/transform_node.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698