From 8c91a8479595d1de04213329e550b467e32bcb20 Mon Sep 17 00:00:00 2001 From: James Seibel Date: Tue, 21 May 2024 07:43:33 -0500 Subject: [PATCH] Fix LZ4 in retail MC --- .../util/objects/dataStreams/DhDataOutputStream.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/core/src/main/java/com/seibel/distanthorizons/core/util/objects/dataStreams/DhDataOutputStream.java b/core/src/main/java/com/seibel/distanthorizons/core/util/objects/dataStreams/DhDataOutputStream.java index 99426b6c0..5fff65581 100644 --- a/core/src/main/java/com/seibel/distanthorizons/core/util/objects/dataStreams/DhDataOutputStream.java +++ b/core/src/main/java/com/seibel/distanthorizons/core/util/objects/dataStreams/DhDataOutputStream.java @@ -56,9 +56,12 @@ public class DhDataOutputStream extends DataOutputStream case LZ4: return new LZ4FrameOutputStream(stream, LZ4FrameOutputStream.BLOCKSIZE.SIZE_64KB, -1L, - // using native instances reduce GC pressure - LZ4Factory.nativeInstance().fastCompressor(), - XXHashFactory.nativeInstance().hash32(), + // using native libraries has the least GC impact, however they also prevent + // shadowJar remapping, so we're going to use the normal fast instance for now + LZ4Factory.fastestInstance().fastCompressor(), + XXHashFactory.fastestInstance().hash32(), + //LZ4Factory.nativeInstance().fastCompressor(), + //XXHashFactory.nativeInstance().hash32(), LZ4FrameOutputStream.FLG.Bits.BLOCK_INDEPENDENCE); case LZMA2: // using an array cache significantly reduces GC pressure