refactor DDC and Analyzer CLI parser option declaration

refactor DDC and Analyzer CLI parser option declaration

R=brianwilkerson@google.com

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

[danrubel, 2016-12-15 18:51:02]

danrubel@google.com changed reviewers:
+ brianwilkerson@google.com

[Brian Wilkerson, 2016-12-15 18:58:40]

lgtm

https://codereview.chromium.org/2580933002/diff/1/pkg/analyzer/lib/src/comman...
File pkg/analyzer/lib/src/command_line/arguments.dart (right):

https://codereview.chromium.org/2580933002/diff/1/pkg/analyzer/lib/src/comman...
pkg/analyzer/lib/src/command_line/arguments.dart:162: * and remove this method.
That should be easy after we get ddc to use the ContextBuilder, but until then
that might be hard. This is a good step toward that.

[danrubel, 2016-12-15 19:02:42]

https://codereview.chromium.org/2580933002/diff/1/pkg/analyzer/lib/src/comman...
File pkg/analyzer/lib/src/command_line/arguments.dart (right):

https://codereview.chromium.org/2580933002/diff/1/pkg/analyzer/lib/src/comman...
pkg/analyzer/lib/src/command_line/arguments.dart:162: * and remove this method.
On 2016/12/15 18:58:40, Brian Wilkerson wrote:
> That should be easy after we get ddc to use the ContextBuilder, but until then
> that might be hard. This is a good step toward that.

Yup. That makes sense as my next step.

[Brian Wilkerson, 2016-12-15 19:09:43]

That's likely to be more than one step :-)

[danrubel, 2016-12-15 19:10:13]

Description was changed from

==========
refactor DDC and Analyzer CLI parser option declaration
==========

to

==========
refactor DDC and Analyzer CLI parser option declaration

R=brianwilkerson@google.com

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

[danrubel, 2016-12-15 19:10:14]

Committed patchset #2 (id:20001) manually as
a105d89e2b07a1799366982db1b3f37b27f37af0 (presubmit successful).