From e9e657cd6e27ffad2ac5b0b250102a9ead04f388 Mon Sep 17 00:00:00 2001 From: Julia Beliaeva Date: Mon, 30 Jan 2017 19:26:58 +0300 Subject: [PATCH] [file-history] use SmartHashSet since most commits have one affected path --- .../com/intellij/vcs/log/data/index/IndexDataGetter.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/platform/vcs-log/impl/src/com/intellij/vcs/log/data/index/IndexDataGetter.java b/platform/vcs-log/impl/src/com/intellij/vcs/log/data/index/IndexDataGetter.java index 97714b603b23..593b90d7889b 100644 --- a/platform/vcs-log/impl/src/com/intellij/vcs/log/data/index/IndexDataGetter.java +++ b/platform/vcs-log/impl/src/com/intellij/vcs/log/data/index/IndexDataGetter.java @@ -21,6 +21,7 @@ import com.intellij.openapi.util.UnorderedPair; import com.intellij.openapi.vcs.FilePath; import com.intellij.openapi.vfs.VirtualFile; import com.intellij.util.containers.ContainerUtil; +import com.intellij.util.containers.SmartHashSet; import com.intellij.util.indexing.StorageException; import com.intellij.vcs.log.impl.FatalErrorHandler; import com.intellij.vcsUtil.VcsUtil; @@ -107,7 +108,7 @@ public class IndexDataGetter { private void addPath(int commit, @NotNull FilePath path) { Set paths = myCommitsToPaths.get(commit); if (paths == null) { - paths = ContainerUtil.newHashSet(); + paths = new SmartHashSet<>(); myCommitsToPaths.put(commit, paths); } paths.add(path); @@ -116,7 +117,7 @@ public class IndexDataGetter { private void addRename(int commit, @NotNull Couple path) { Set> paths = myCommitsToRenames.get(commit); if (paths == null) { - paths = ContainerUtil.newHashSet(); + paths = new SmartHashSet<>(); myCommitsToRenames.put(commit, paths); } paths.add(new UnorderedPair<>(path.first, path.second)); @@ -178,7 +179,7 @@ public class IndexDataGetter { if (paths != null) return paths; Set> renames = myCommitsToRenames.get(commit); - Set result = ContainerUtil.newHashSet(); + Set result = new SmartHashSet<>(); for (UnorderedPair rename : renames) { result.add(rename.first); result.add(rename.second);