From 6d6d2060afacccfa940313375ffbeb951e7c50b6 Mon Sep 17 00:00:00 2001 From: Aayush Atharva Date: Thu, 5 Aug 2021 17:47:00 +0530 Subject: [PATCH] Remove unnecessary toString calls (#11550) Motivation: We should get rid of the unnecessary toString calls because they're redundant in nature. Modification: Removed unnecessary toString calls. Result: Better code --- .../handler/codec/http/HttpResponseEncoderTest.java | 8 ++++---- .../http2/Http2StreamFrameToHttpObjectCodec.java | 2 +- .../io/netty/handler/codec/mqtt/MqttDecoder.java | 12 ++++++------ .../handler/codec/stomp/StompSubframeDecoder.java | 4 ++-- 4 files changed, 13 insertions(+), 13 deletions(-) diff --git a/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseEncoderTest.java b/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseEncoderTest.java index 09338489d7..3ecd918a90 100644 --- a/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseEncoderTest.java +++ b/codec-http/src/test/java/io/netty/handler/codec/http/HttpResponseEncoderTest.java @@ -286,14 +286,14 @@ public class HttpResponseEncoderTest { ByteBuf buffer = channel.readOutbound(); StringBuilder responseText = new StringBuilder(); - responseText.append(HttpVersion.HTTP_1_1.toString()).append(' ').append(status.toString()).append("\r\n"); + responseText.append(HttpVersion.HTTP_1_1).append(' ').append(status.toString()).append("\r\n"); if (!headerStripped && headerName != null) { responseText.append(headerName).append(": "); if (HttpHeaderNames.CONTENT_LENGTH.contentEquals(headerName)) { responseText.append('0'); } else { - responseText.append(HttpHeaderValues.CHUNKED.toString()); + responseText.append(HttpHeaderValues.CHUNKED); } responseText.append("\r\n"); } @@ -383,8 +383,8 @@ public class HttpResponseEncoderTest { assertTrue(channel.writeOutbound(LastHttpContent.EMPTY_LAST_CONTENT)); StringBuilder responseText = new StringBuilder(); - responseText.append(HttpVersion.HTTP_1_1.toString()).append(' ') - .append(HttpResponseStatus.RESET_CONTENT.toString()).append("\r\n"); + responseText.append(HttpVersion.HTTP_1_1).append(' ') + .append(HttpResponseStatus.RESET_CONTENT).append("\r\n"); responseText.append(HttpHeaderNames.CONTENT_LENGTH).append(": 0\r\n"); responseText.append("\r\n"); diff --git a/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2StreamFrameToHttpObjectCodec.java b/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2StreamFrameToHttpObjectCodec.java index b5784495a0..0eb0c7a5de 100644 --- a/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2StreamFrameToHttpObjectCodec.java +++ b/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2StreamFrameToHttpObjectCodec.java @@ -162,7 +162,7 @@ public class Http2StreamFrameToHttpObjectCodec extends MessageToMessageCodec { case SUBSCRIBE: case UNSUBSCRIBE: if (dupFlag) { - throw new DecoderException("Illegal BIT 3 in fixed header of " + messageType.toString() + throw new DecoderException("Illegal BIT 3 in fixed header of " + messageType + " message, must be 0, found 1"); } if (qosLevel != 1) { - throw new DecoderException("Illegal QOS Level in fixed header of " + messageType.toString() + throw new DecoderException("Illegal QOS Level in fixed header of " + messageType + " message, must be 1, found " + qosLevel); } if (retain) { - throw new DecoderException("Illegal BIT 0 in fixed header of " + messageType.toString() + throw new DecoderException("Illegal BIT 0 in fixed header of " + messageType + " message, must be 0, found 1"); } break; @@ -207,15 +207,15 @@ public final class MqttDecoder extends ReplayingDecoder { case SUBACK: case UNSUBACK: if (dupFlag) { - throw new DecoderException("Illegal BIT 3 in fixed header of " + messageType.toString() + throw new DecoderException("Illegal BIT 3 in fixed header of " + messageType + " message, must be 0, found 1"); } if (qosLevel != 0) { - throw new DecoderException("Illegal BIT 2 or 1 in fixed header of " + messageType.toString() + throw new DecoderException("Illegal BIT 2 or 1 in fixed header of " + messageType + " message, must be 0, found " + qosLevel); } if (retain) { - throw new DecoderException("Illegal BIT 0 in fixed header of " + messageType.toString() + throw new DecoderException("Illegal BIT 0 in fixed header of " + messageType + " message, must be 0, found 1"); } break; diff --git a/codec-stomp/src/main/java/io/netty/handler/codec/stomp/StompSubframeDecoder.java b/codec-stomp/src/main/java/io/netty/handler/codec/stomp/StompSubframeDecoder.java index 107b8589c2..567f7c87cd 100644 --- a/codec-stomp/src/main/java/io/netty/handler/codec/stomp/StompSubframeDecoder.java +++ b/codec-stomp/src/main/java/io/netty/handler/codec/stomp/StompSubframeDecoder.java @@ -343,9 +343,9 @@ public class StompSubframeDecoder extends ReplayingDecoder { headers.add(name, value.toString()); } else if (validateHeaders) { if (StringUtil.isNullOrEmpty(name)) { - throw new IllegalArgumentException("received an invalid header line '" + value.toString() + '\''); + throw new IllegalArgumentException("received an invalid header line '" + value + '\''); } - String line = name + ':' + value.toString(); + String line = name + ':' + value; throw new IllegalArgumentException("a header value or name contains a prohibited character ':'" + ", " + line); }