Analyze priority files first.

Analyze priority files first.

TBR

R=brianwilkerson@google.com, paulberry@google.com
BUG=

Committed: https://github.com/dart-lang/sdk/commit/fcce68e2dfdb8b9942d04431c381bce95444b723

[scheglov, 2016-10-26 04:44:53]

Description was changed from

==========
Analyze priority files first.

TBR

R=brianwilkerson@google.com, paulberry@google.com
BUG=
==========

to

==========
Analyze priority files first.

TBR

R=brianwilkerson@google.com, paulberry@google.com
BUG=

Committed:
https://github.com/dart-lang/sdk/commit/fcce68e2dfdb8b9942d04431c381bce95444b723
==========

[scheglov, 2016-10-26 04:44:54]

Committed patchset #1 (id:1) manually as
fcce68e2dfdb8b9942d04431c381bce95444b723 (presubmit successful).

[Brian Wilkerson, 2016-10-26 08:16:54]

lgtm

[Paul Berry, 2016-10-31 16:51:40]

https://codereview.chromium.org/2453693002/diff/1/pkg/analyzer/lib/src/dart/a...
File pkg/analyzer/lib/src/dart/analysis/driver.dart (right):

https://codereview.chromium.org/2453693002/diff/1/pkg/analyzer/lib/src/dart/a...
pkg/analyzer/lib/src/dart/analysis/driver.dart:240: break;
Do need to set `analyzed = true` here?  Otherwise it looks like the code below
(lines 245 and 246) will be unreachable.

[scheglov, 2016-10-31 17:38:52]

https://codereview.chromium.org/2453693002/diff/1/pkg/analyzer/lib/src/dart/a...
File pkg/analyzer/lib/src/dart/analysis/driver.dart (right):

https://codereview.chromium.org/2453693002/diff/1/pkg/analyzer/lib/src/dart/a...
pkg/analyzer/lib/src/dart/analysis/driver.dart:240: break;
On 2016/10/31 16:51:40, Paul Berry wrote:
> Do need to set `analyzed = true` here?  Otherwise it looks like the code below
> (lines 245 and 246) will be unreachable.

Yes, you are right, we need to set `analyzed = true`.
I noticed and fixed this later in one of the CLs.