From d97e6aeb2d14d3656614b14b378cc08cd158c8b0 Mon Sep 17 00:00:00 2001 From: peter Date: Wed, 29 May 2019 18:24:57 +0200 Subject: [PATCH] don't compute expensive content roots for PSI event where nobody needs them (IDEA-204030) GitOrigin-RevId: b723b6d57f2fb807ece7907695adc0b2e6212e42 --- .../intellij/psi/impl/file/impl/PsiVFSListener.java | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/platform/lang-impl/src/com/intellij/psi/impl/file/impl/PsiVFSListener.java b/platform/lang-impl/src/com/intellij/psi/impl/file/impl/PsiVFSListener.java index cd6bcc38d213..687e92530eff 100644 --- a/platform/lang-impl/src/com/intellij/psi/impl/file/impl/PsiVFSListener.java +++ b/platform/lang-impl/src/com/intellij/psi/impl/file/impl/PsiVFSListener.java @@ -561,7 +561,6 @@ public class PsiVFSListener implements BulkFileListener { } private class MyModuleRootListener implements ModuleRootListener { - private VirtualFile[] myOldContentRoots; private int depthCounter; // accessed from within write action only @Override public void beforeRootsChange(@NotNull final ModuleRootEvent event) { @@ -573,10 +572,6 @@ public class PsiVFSListener implements BulkFileListener { PsiTreeChangeEventImpl treeEvent = new PsiTreeChangeEventImpl(myManager); treeEvent.setPropertyName(PsiTreeChangeEvent.PROP_ROOTS); - final VirtualFile[] contentRoots = myProjectRootManager.getContentRoots(); - LOG.assertTrue(myOldContentRoots == null); - myOldContentRoots = contentRoots; - treeEvent.setOldValue(contentRoots); myManager.beforePropertyChange(treeEvent); } ); @@ -597,11 +592,6 @@ public class PsiVFSListener implements BulkFileListener { PsiTreeChangeEventImpl treeEvent = new PsiTreeChangeEventImpl(myManager); treeEvent.setPropertyName(PsiTreeChangeEvent.PROP_ROOTS); - final VirtualFile[] contentRoots = myProjectRootManager.getContentRoots(); - treeEvent.setNewValue(contentRoots); - LOG.assertTrue(myOldContentRoots != null); - treeEvent.setOldValue(myOldContentRoots); - myOldContentRoots = null; myManager.propertyChanged(treeEvent); } );