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

Issue 16357010: Basic dart^2 analyzer tests running. (Closed)

Created:
7 years, 6 months ago by scheglov
Modified:
7 years, 6 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org, ricow1
Visibility:
Public.

Description

Basic dart^2 analyzer tests running. Status file only for co19 now, more status files in next CL. R=brianwilkerson@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=23602

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+98 lines, -38 lines) Patch
M editor/util/plugins/com.google.dart.java2dart/resources/java_io.dart View 1 chunk +11 lines, -2 lines 0 comments Download
M pkg/analyzer_experimental/bin/analyzer.dart View 4 chunks +9 lines, -4 lines 0 comments Download
M pkg/analyzer_experimental/lib/options.dart View 3 chunks +5 lines, -4 lines 0 comments Download
M pkg/analyzer_experimental/lib/src/analyzer_impl.dart View 2 chunks +6 lines, -2 lines 0 comments Download
M pkg/analyzer_experimental/lib/src/error_formatter.dart View 2 chunks +2 lines, -7 lines 2 comments Download
M pkg/analyzer_experimental/lib/src/generated/ast.dart View 1 chunk +1 line, -1 line 0 comments Download
M pkg/analyzer_experimental/lib/src/generated/constant.dart View 1 chunk +2 lines, -4 lines 0 comments Download
M pkg/analyzer_experimental/lib/src/generated/java_io.dart View 1 chunk +11 lines, -2 lines 0 comments Download
A sdk/bin/dart2analyzer View 1 chunk +33 lines, -0 lines 0 comments Download
A + sdk/bin/dart2analyzer_developer View 1 chunk +2 lines, -5 lines 0 comments Download
A + tests/co19/co19-analyzer2.status View 2 chunks +4 lines, -4 lines 0 comments Download
M tests/co19/test_config.dart View 1 chunk +1 line, -0 lines 0 comments Download
M tools/testing/dart/test_options.dart View 3 chunks +5 lines, -2 lines 0 comments Download
M tools/testing/dart/test_suite.dart View 5 chunks +6 lines, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
scheglov
7 years, 6 months ago (2013-06-04 15:23:44 UTC) #1
Brian Wilkerson
LGTM. Ideally we wouldn't need a separate status file for it, but I guess we're ...
7 years, 6 months ago (2013-06-04 15:56:47 UTC) #2
scheglov
Yes, it is not possible to share status files yet. https://codereview.chromium.org/16357010/diff/1/pkg/analyzer_experimental/lib/src/error_formatter.dart File pkg/analyzer_experimental/lib/src/error_formatter.dart (right): https://codereview.chromium.org/16357010/diff/1/pkg/analyzer_experimental/lib/src/error_formatter.dart#newcode126 ...
7 years, 6 months ago (2013-06-04 16:08:52 UTC) #3
scheglov
Committed patchset #1 manually as r23602 (presubmit successful).
7 years, 6 months ago (2013-06-04 16:09:23 UTC) #4
ahe
What is this change about? Why are you making changes to the test infrastructure without ...
7 years, 6 months ago (2013-06-10 07:20:13 UTC) #5
ahe
On 2013/06/10 07:20:13, ahe wrote: > What is this change about? > > Why are ...
7 years, 6 months ago (2013-06-10 07:20:50 UTC) #6
kustermann
7 years, 6 months ago (2013-06-10 07:57:07 UTC) #7
Message was sent while issue was closed.
On 2013/06/10 07:20:50, ahe wrote:
> On 2013/06/10 07:20:13, ahe wrote:
> > What is this change about?
> > 
> > Why are you making changes to the test infrastructure without coordinating
> with
> > the test team?
> > 
> > Are you planning to ship sdk/bin/dart2analyzer as part of the SDK, if so why
> do
> > we need to analyzers in the SDK?
> 
> I meant: "why do we need to analyzers in the SDK?"

I wasn't aware of this CL either.
I guess you want to move slowly over to the java2dart'ed analyzer and want to
get testing support for it?
We could probably also setup a builder on FYI for this.

One DBC about the CL: I don't see a real reason why you need to duplicate the
status file -- the differences are pretty small.

Powered by Google App Engine
This is Rietveld 408576698