andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

MqttConnectPayload.toString() should use Arrays.toString() instead of [].toString() (#9292)

Browse Source 
Motivation:

The toString() method should use Arrays.toString() to produce a meaningful String representation for arrays.

Modification:

Use Arrays.toString()

Result:

More useful toString() implementation
This commit is contained in:
jimin 2019-06-28 03:55:02 +08:00 committed by Norman Maurer
parent a469c2eaac
commit 757e48c3e9
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
codec-mqtt/src/main/java/io/netty/handler/codec/mqtt/MqttConnectPayload.java
View File

@ -16,6 +16,8 @@
package io.netty.handler.codec.mqtt; package io.netty.handler.codec.mqtt;
import java.util.Arrays;
import io.netty.util.CharsetUtil; import io.netty.util.CharsetUtil;
import io.netty.util.internal.StringUtil; import io.netty.util.internal.StringUtil;
@ -103,9 +105,9 @@ public final class MqttConnectPayload {
.append('[') .append('[')
.append("clientIdentifier=").append(clientIdentifier) .append("clientIdentifier=").append(clientIdentifier)
.append(", willTopic=").append(willTopic) .append(", willTopic=").append(willTopic)
.append(", willMessage=").append(willMessage) .append(", willMessage=").append(Arrays.toString(willMessage))
.append(", userName=").append(userName) .append(", userName=").append(userName)
.append(", password=").append(password) .append(", password=").append(Arrays.toString(password))
.append(']') .append(']')
.toString(); .toString();
} }