From 7a82d4a283b7ffd4b13a4bf8994198b68620f23b Mon Sep 17 00:00:00 2001
From: "Maxim.Mossienko" <Maxim.Mossienko@jetbrains.com>
Date: Mon, 17 Oct 2016 16:55:53 +0200
Subject: [PATCH] avoid extra copying data during append

---
 .../com/intellij/util/indexing/ValueContainerMap.java    | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/platform/lang-impl/src/com/intellij/util/indexing/ValueContainerMap.java b/platform/lang-impl/src/com/intellij/util/indexing/ValueContainerMap.java
index 7350f2bc7d80..e6789513ee35 100644
--- a/platform/lang-impl/src/com/intellij/util/indexing/ValueContainerMap.java
+++ b/platform/lang-impl/src/com/intellij/util/indexing/ValueContainerMap.java
@@ -15,9 +15,7 @@
  */
 package com.intellij.util.indexing;
 
-import com.intellij.openapi.util.io.BufferExposingByteArrayOutputStream;
 import com.intellij.util.io.DataExternalizer;
-import com.intellij.util.io.DataOutputStream;
 import com.intellij.util.io.KeyDescriptor;
 import com.intellij.util.io.PersistentHashMap;
 import org.jetbrains.annotations.NotNull;
@@ -56,15 +54,10 @@ class ValueContainerMap<Key, Value> extends PersistentHashMap<Key, ValueContaine
       // note that keys unique for indexed file have their value calculated at once (e.g. key is file id, index calculates something for particular
       // file) and there is no benefit to accumulate values for particular key because only one value exists
       if (!valueContainer.needsCompacting() && !myKeyIsUniqueForIndexedFile) {
-        final BufferExposingByteArrayOutputStream bytes = new BufferExposingByteArrayOutputStream();
-        //noinspection IOResourceOpenedButNotSafelyClosed
-        final DataOutputStream _out = new DataOutputStream(bytes);
-        valueContainer.saveTo(_out, myValueExternalizer);
-
         appendData(key, new PersistentHashMap.ValueDataAppender() {
           @Override
           public void append(@NotNull final DataOutput out) throws IOException {
-            out.write(bytes.getInternalBuffer(), 0, bytes.size());
+            valueContainer.saveTo(out, myValueExternalizer);
           }
         });
       }