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

Issue 2733953005: Correct --persistent_worker arg constraint check (Closed)

Created:
3 years, 9 months ago by Nate Bosch
Modified:
3 years, 9 months ago
Reviewers:
jakemac, danrubel
CC:
reviews_dartlang.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Correct --persistent_worker arg constraint check Previously the args were normalized to change an arg with `=` into separate args. That normalization was removed in https://github.com/dart-lang/sdk/commit/e321439470ec016e9532afe57b7f2891c0282d83 This fixes an issue with using the analyzer in worker mode on the current SDK. - Allow the case where there are 3 args and one of them starts with --dart-sdk rather than require it to be the entire string - Split into two bools with more descriptive names and refactor from an if/else chain into a composed condition R=jakemac@google.com Committed: https://github.com/dart-lang/sdk/commit/56ad714d6b179839f1ced976bd5d5500f814a482

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -11 lines) Patch
M pkg/analyzer_cli/lib/src/options.dart View 1 chunk +5 lines, -11 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
Nate Bosch
3 years, 9 months ago (2017-03-08 00:31:52 UTC) #2
jakemac
lgtm
3 years, 9 months ago (2017-03-08 15:24:08 UTC) #3
Nate Bosch
3 years, 9 months ago (2017-03-08 20:10:31 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
56ad714d6b179839f1ced976bd5d5500f814a482 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698