From 3cffe08a1cd3af4fcf4a36025560491bc0ae0a44 Mon Sep 17 00:00:00 2001 From: Santiago Pericasgeertsen Date: Tue, 18 Jun 2019 13:03:12 -0400 Subject: [PATCH] Use remaining() instead of limit(). Remove unnecessary call to release(). (#795) Signed-off-by: Santiago Pericas-Geertsen --- .../src/main/java/io/helidon/webserver/BareResponseImpl.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/webserver/webserver/src/main/java/io/helidon/webserver/BareResponseImpl.java b/webserver/webserver/src/main/java/io/helidon/webserver/BareResponseImpl.java index 234ba7d8354..063efc8ca65 100644 --- a/webserver/webserver/src/main/java/io/helidon/webserver/BareResponseImpl.java +++ b/webserver/webserver/src/main/java/io/helidon/webserver/BareResponseImpl.java @@ -224,7 +224,7 @@ private void writeLastContent(final Throwable throwable, final ChannelFutureList if (lengthOptimization) { if (firstChunk != null) { HttpUtil.setTransferEncodingChunked(response, false); - HttpUtil.setContentLength(response, firstChunk.data().limit()); + HttpUtil.setContentLength(response, firstChunk.data().remaining()); } initWriteResponse(); } @@ -295,7 +295,6 @@ private ChannelFuture initWriteResponse() { response = null; if (firstChunk != null) { cf = sendData(firstChunk); - firstChunk.release(); firstChunk = null; } lengthOptimization = false;