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

Issue 542193005: Delete stale compiled files. (Closed)

Created:
6 years, 3 months ago by Bob Nystrom
Modified:
6 years, 3 months ago
Reviewers:
nweiz
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Delete stale compiled files. BUG=https://code.google.com/p/dart/issues/detail?id=20873 R=nweiz@google.com Committed: https://code.google.com/p/dart/source/detail?r=40134

Patch Set 1 #

Total comments: 4

Patch Set 2 : Revise! #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -4 lines) Patch
M sdk/lib/_internal/pub/bin/async_compile.dart View 1 2 chunks +17 lines, -2 lines 0 comments Download
M sdk/lib/_internal/pub_generated/bin/async_compile.dart View 2 chunks +12 lines, -1 line 0 comments Download
M sdk/lib/_internal/pub_generated/lib/src/barback/asset_environment.dart View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 5 (1 generated)
Bob Nystrom
6 years, 3 months ago (2014-09-10 20:29:40 UTC) #2
nweiz
lgtm https://codereview.chromium.org/542193005/diff/1/sdk/lib/_internal/pub/bin/async_compile.dart File sdk/lib/_internal/pub/bin/async_compile.dart (right): https://codereview.chromium.org/542193005/diff/1/sdk/lib/_internal/pub/bin/async_compile.dart#newcode98 sdk/lib/_internal/pub/bin/async_compile.dart:98: var sources = new Set<String>(); Nit: I wouldn't ...
6 years, 3 months ago (2014-09-10 21:08:18 UTC) #3
Bob Nystrom
https://codereview.chromium.org/542193005/diff/1/sdk/lib/_internal/pub/bin/async_compile.dart File sdk/lib/_internal/pub/bin/async_compile.dart (right): https://codereview.chromium.org/542193005/diff/1/sdk/lib/_internal/pub/bin/async_compile.dart#newcode98 sdk/lib/_internal/pub/bin/async_compile.dart:98: var sources = new Set<String>(); On 2014/09/10 21:08:18, nweiz ...
6 years, 3 months ago (2014-09-10 22:42:34 UTC) #4
Bob Nystrom
6 years, 3 months ago (2014-09-11 00:33:33 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 (id:20001) manually as 40134 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698