From f2596fd993f7036a93c796d577deb5df49dd3958 Mon Sep 17 00:00:00 2001 From: Norman Maurer Date: Wed, 27 Nov 2019 06:54:55 +0100 Subject: [PATCH] Simplify Deflate* implementations by using EmbeddedChannel.finishAndReleaseAll() (#9808) Motivation: We can simplify the code by just using finishAndReleaseAll() Modifications: Remove some code and simplify Result: Cleaner code --- .../extensions/compression/DeflateDecoder.java | 11 +---------- .../extensions/compression/DeflateEncoder.java | 11 +---------- 2 files changed, 2 insertions(+), 20 deletions(-) diff --git a/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateDecoder.java b/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateDecoder.java index 223e2e6520..5b70620b62 100644 --- a/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateDecoder.java +++ b/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateDecoder.java @@ -150,16 +150,7 @@ abstract class DeflateDecoder extends WebSocketExtensionDecoder { private void cleanup() { if (decoder != null) { // Clean-up the previous encoder if not cleaned up correctly. - if (decoder.finish()) { - for (;;) { - ByteBuf buf = decoder.readOutbound(); - if (buf == null) { - break; - } - // Release the buffer - buf.release(); - } - } + decoder.finishAndReleaseAll(); decoder = null; } } diff --git a/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateEncoder.java b/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateEncoder.java index 1203c49880..4f3daa81c1 100644 --- a/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateEncoder.java +++ b/codec-http/src/main/java/io/netty/handler/codec/http/websocketx/extensions/compression/DeflateEncoder.java @@ -158,16 +158,7 @@ abstract class DeflateEncoder extends WebSocketExtensionEncoder { private void cleanup() { if (encoder != null) { // Clean-up the previous encoder if not cleaned up correctly. - if (encoder.finish()) { - for (;;) { - ByteBuf buf = encoder.readOutbound(); - if (buf == null) { - break; - } - // Release the buffer - buf.release(); - } - } + encoder.finishAndReleaseAll(); encoder = null; } }