From 1247257fe08973fbbd0911ee1f94c7cc2d24f3a6 Mon Sep 17 00:00:00 2001 From: "Andrei.Kuznetsov" Date: Fri, 11 Oct 2024 12:22:52 +0200 Subject: [PATCH] Don't log psiLen in VfsEventsMerger, because there is no fast and reliable way to understand if file may have PSI or not GitOrigin-RevId: f0de712a22a10d2ec2bff34a7d9798f320556bf6 --- .../com/intellij/util/indexing/events/VfsEventsMerger.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/platform/indexing-impl/src/com/intellij/util/indexing/events/VfsEventsMerger.java b/platform/indexing-impl/src/com/intellij/util/indexing/events/VfsEventsMerger.java index 9bf2ef26dc07..00ab1db5c105 100644 --- a/platform/indexing-impl/src/com/intellij/util/indexing/events/VfsEventsMerger.java +++ b/platform/indexing-impl/src/com/intellij/util/indexing/events/VfsEventsMerger.java @@ -261,7 +261,10 @@ public final class VfsEventsMerger { if (indexedFile instanceof FileContent fileContent) { extra += ",contLen(b)=" + fileContent.getContent().length; FileType fileType = fileContent.getFileType(); - extra += ",psiLen=" + (fileType instanceof LanguageFileType ? fileContent.getPsiFile().getTextLength() : -1); + // WARNING: LanguageFileType does not guarantee that there is a PsiFile. + // Example: org.jetbrains.bazel.languages.projectview.base.ProjectViewFileType + // psiLen has never been helpful to me, so don't log it for now. + // extra += ",psiLen=" + (fileType instanceof LanguageFileType ? fileContent.getPsiFile().getTextLength() : -1); extra += ",bin=" + (fileType.isBinary() ? "t" : "f"); }