Include 'enableGenericMethods' into salt.

Include 'enableGenericMethods' into salt.

R=brianwilkerson@google.com
BUG=

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

[Brian Wilkerson, 2016-11-08 21:15:08]

lgtm

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

https://codereview.chromium.org/2485853003/diff/1/pkg/analyzer/lib/src/dart/a...
pkg/analyzer/lib/src/dart/analysis/driver.dart:634: analysisOptionsSalt |=
_analysisOptions.enableGenericMethods ? (1 << 1) : 0;
Have you looked at AnalysisOptionsImpl.encodeCrossContextOptions()? There are
several other options that affect analysis, and there's already an encoding for
them.

[scheglov, 2016-11-09 01:21:38]

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

https://codereview.chromium.org/2485853003/diff/1/pkg/analyzer/lib/src/dart/a...
pkg/analyzer/lib/src/dart/analysis/driver.dart:634: analysisOptionsSalt |=
_analysisOptions.enableGenericMethods ? (1 << 1) : 0;
On 2016/11/08 21:15:08, Brian Wilkerson wrote:
> Have you looked at AnalysisOptionsImpl.encodeCrossContextOptions()? There are
> several other options that affect analysis, and there's already an encoding
for
> them.

Done.

[scheglov, 2016-11-09 01:24:11]

Description was changed from

==========
Include 'enableGenericMethods' into salt.

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

to

==========
Include 'enableGenericMethods' into salt.

R=brianwilkerson@google.com
BUG=

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

[scheglov, 2016-11-09 01:24:13]

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