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

Issue 2183003003: Associate excludes with context data. (Closed)

Created:
4 years, 4 months ago by pquitslund
Modified:
4 years, 4 months ago
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Associate excludes with context data. A step towards centralizing file exclusion handling (which is currently duplicated by all analyzer clients). BUG= R=scheglov@google.com Committed: https://github.com/dart-lang/sdk/commit/065238943adbb21e43d2db5c7e2e25b90b870947

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+70 lines, -33 lines) Patch
M pkg/analysis_server/lib/src/context_manager.dart View 3 chunks +2 lines, -20 lines 0 comments Download
M pkg/analyzer/lib/src/context/context.dart View 2 chunks +8 lines, -0 lines 4 comments Download
M pkg/analyzer/lib/src/task/options.dart View 4 chunks +28 lines, -13 lines 0 comments Download
M pkg/analyzer/lib/src/util/yaml.dart View 1 chunk +19 lines, -0 lines 0 comments Download
M pkg/analyzer/test/src/task/options_test.dart View 2 chunks +13 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
pquitslund
4 years, 4 months ago (2016-07-26 17:14:38 UTC) #2
scheglov
LGTM https://codereview.chromium.org/2183003003/diff/1/pkg/analyzer/lib/src/context/context.dart File pkg/analyzer/lib/src/context/context.dart (right): https://codereview.chromium.org/2183003003/diff/1/pkg/analyzer/lib/src/context/context.dart#newcode48 pkg/analyzer/lib/src/context/context.dart:48: final ResultDescriptor<List<String>> CONTEXT_EXCLUDES = ListResultDescriptor<String> https://codereview.chromium.org/2183003003/diff/1/pkg/analyzer/lib/src/context/context.dart#newcode49 pkg/analyzer/lib/src/context/context.dart:49: new ...
4 years, 4 months ago (2016-07-26 17:21:20 UTC) #3
pquitslund
Thanks! https://codereview.chromium.org/2183003003/diff/1/pkg/analyzer/lib/src/context/context.dart File pkg/analyzer/lib/src/context/context.dart (right): https://codereview.chromium.org/2183003003/diff/1/pkg/analyzer/lib/src/context/context.dart#newcode48 pkg/analyzer/lib/src/context/context.dart:48: final ResultDescriptor<List<String>> CONTEXT_EXCLUDES = On 2016/07/26 17:21:20, scheglov ...
4 years, 4 months ago (2016-07-26 17:24:15 UTC) #4
pquitslund
Committed patchset #1 (id:1) manually as 065238943adbb21e43d2db5c7e2e25b90b870947 (presubmit successful).
4 years, 4 months ago (2016-07-26 17:24:46 UTC) #6
Brian Wilkerson
4 years, 4 months ago (2016-07-27 14:52:09 UTC) #7
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698