From 80645c519a5aae4edc562a6dbc4c3cc53361ae69 Mon Sep 17 00:00:00 2001 From: Yongho Choi Date: Tue, 5 Jul 2016 16:59:47 +0900 Subject: [PATCH] Added break in switch case. I guess this things to make a mistake if you skip. --- .../com/corundumstudio/socketio/handler/PacketListener.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/main/java/com/corundumstudio/socketio/handler/PacketListener.java b/src/main/java/com/corundumstudio/socketio/handler/PacketListener.java index c3583ff3e..8be1e287d 100644 --- a/src/main/java/com/corundumstudio/socketio/handler/PacketListener.java +++ b/src/main/java/com/corundumstudio/socketio/handler/PacketListener.java @@ -80,6 +80,7 @@ public void onPacket(Packet packet, NamespaceClient client, Transport transport) if (packet.getSubType() == PacketType.DISCONNECT) { client.onDisconnect(); + break; } if (packet.getSubType() == PacketType.CONNECT) { @@ -87,10 +88,12 @@ public void onPacket(Packet packet, NamespaceClient client, Transport transport) namespace.onConnect(client); // send connect handshake packet back to client client.getBaseClient().send(packet, transport); + break; } if (packet.getSubType() == PacketType.ACK) { ackManager.onAck(client, packet); + break; } if (packet.getSubType() == PacketType.EVENT @@ -101,6 +104,7 @@ public void onPacket(Packet packet, NamespaceClient client, Transport transport) args = packet.getData(); } namespace.onEvent(client, packet.getName(), args, ackRequest); + break; } break; }