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

Unified Diff: pkg/analysis_server/lib/src/operation/operation_analysis.dart

Issue 2980653002: Don't include *.yaml files in 'analysis.analyzedFiles' notification. (Closed)
Patch Set: Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | pkg/analysis_server/test/analysis/notification_analyzedFiles_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/analysis_server/lib/src/operation/operation_analysis.dart
diff --git a/pkg/analysis_server/lib/src/operation/operation_analysis.dart b/pkg/analysis_server/lib/src/operation/operation_analysis.dart
index a53200505c5ebbebe6c3c443322ace769b98b940..0b57104f3a5c6c22ead47c8e2265354cad423a4c 100644
--- a/pkg/analysis_server/lib/src/operation/operation_analysis.dart
+++ b/pkg/analysis_server/lib/src/operation/operation_analysis.dart
@@ -67,6 +67,11 @@ void sendAnalysisNotificationAnalyzedFiles(AnalysisServer server) {
.map((driver) => driver.knownFiles)
.expand((files) => files)
.toSet();
+
+ // Exclude *.yaml files because IDEA Dart plugin attempts to index
+ // all the files in folders which contain analyzed files.
+ analyzedFiles.removeWhere((file) => file.endsWith('.yaml'));
+
Set<String> prevAnalyzedFiles = server.prevAnalyzedFiles;
if (prevAnalyzedFiles != null &&
prevAnalyzedFiles.length == analyzedFiles.length &&
« no previous file with comments | « no previous file | pkg/analysis_server/test/analysis/notification_analyzedFiles_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698