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

Issue 1372173004: Ignore added files in hidden directories. (Closed)

Created:
5 years, 2 months ago by Paul Berry
Modified:
5 years, 2 months ago
Reviewers:
Brian Wilkerson
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Ignore added files in hidden directories. In commit 6ddc37a5cc402520da5d38695573cc335b8e8a35, the ContextManager was modified to disable analysis of files in hidden directories, provided those files existed prior to the creation of the context. This CL extends the fix to apply also to files that are added after context creation. Fixes #24212. R=brianwilkerson@google.com Committed: https://github.com/dart-lang/sdk/commit/f9aebddc67191b9c136ddff2e95db6869ce1b569

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+75 lines, -1 line) Patch
M pkg/analysis_server/lib/src/context_manager.dart View 2 chunks +18 lines, -1 line 0 comments Download
M pkg/analysis_server/test/context_manager_test.dart View 3 chunks +57 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (1 generated)
Paul Berry
5 years, 2 months ago (2015-09-28 20:26:35 UTC) #2
Brian Wilkerson
LGTM
5 years, 2 months ago (2015-09-28 20:33:07 UTC) #3
Paul Berry
5 years, 2 months ago (2015-09-28 21:10:39 UTC) #4
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
f9aebddc67191b9c136ddff2e95db6869ce1b569 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698