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

Issue 21410002: Fixes analyzer status file. (Closed)

Created:
7 years, 4 months ago by zra
Modified:
7 years, 4 months ago
Reviewers:
siva
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fixes analyzer status file. Why do we have to duplicate this information across status files? Wouldn't it be easier for the analyzer test scripts to use the existing status files? Committed: https://code.google.com/p/dart/source/detail?r=25684

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M tests/lib/analyzer/analyze_tests.status View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
zra
TBR
7 years, 4 months ago (2013-07-31 21:46:56 UTC) #1
zra
Committed patchset #1 manually as r25684 (presubmit successful).
7 years, 4 months ago (2013-07-31 21:47:11 UTC) #2
zra
7 years, 4 months ago (2013-07-31 21:47:41 UTC) #3
siva
7 years, 4 months ago (2013-07-31 22:10:14 UTC) #4
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698