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

Change String comparison to equals() from == (#10022)

Browse Source 
Motivation:

Even if it was stored in the string constant pool, I thought it was safe to compare it through the Equals() method.

Modification:

So, I changed "==" comparison to equals() comparison

Result:

It has become safer to compare String values with different references and with the same values.
This commit is contained in:
Hyunjin Choi 2020-02-13 17:57:14 +09:00 committed by Norman Maurer
parent 064ab7afa8
commit 6980e0de65
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
codec-http2/src/test/java/io/netty/handler/codec/http2/LastInboundHandler.java
View File

@ -94,7 +94,7 @@ public class LastInboundHandler implements ChannelHandler {
@Override @Override
public void channelWritabilityChanged(ChannelHandlerContext ctx) throws Exception { public void channelWritabilityChanged(ChannelHandlerContext ctx) throws Exception {
if (writabilityStates == "") { if ("".equals(writabilityStates)) {
writabilityStates = String.valueOf(ctx.channel().isWritable()); writabilityStates = String.valueOf(ctx.channel().isWritable());
} else { } else {
writabilityStates += "," + ctx.channel().isWritable(); writabilityStates += "," + ctx.channel().isWritable();