Browse Source

Unreleased butebuf logging

master
Nikita 12 years ago
parent
commit
309fe9a22e
  1. 5
      src/main/java/com/corundumstudio/socketio/handler/EncoderHandler.java
  2. 3
      src/main/java/com/corundumstudio/socketio/parser/Encoder.java

5
src/main/java/com/corundumstudio/socketio/handler/EncoderHandler.java

@ -144,6 +144,11 @@ public class EncoderHandler extends ChannelOutboundHandlerAdapter {
if (msg instanceof WebsocketErrorMessage) { if (msg instanceof WebsocketErrorMessage) {
handle((WebsocketErrorMessage) msg, ctx.channel(), out); handle((WebsocketErrorMessage) msg, ctx.channel(), out);
} }
if (out.refCnt() > 0) {
log.warn("Buf for message: {} has not been deallocated, refCnt: {}! Forcing release it!", msg, out.refCnt());
out.release();
}
} }
private void handle(AuthorizeMessage authMsg, Channel channel, ByteBuf out) throws IOException { private void handle(AuthorizeMessage authMsg, Channel channel, ByteBuf out) throws IOException {

3
src/main/java/com/corundumstudio/socketio/parser/Encoder.java

@ -68,6 +68,7 @@ public class Encoder {
} }
ByteBuf packetBuffer = allocateBuffer(allocator); ByteBuf packetBuffer = allocateBuffer(allocator);
try {
int len = encodePacket(packet, packetBuffer); int len = encodePacket(packet, packetBuffer);
byte[] lenBytes = toChars(len); byte[] lenBytes = toChars(len);
@ -75,10 +76,12 @@ public class Encoder {
buffer.writeBytes(lenBytes); buffer.writeBytes(lenBytes);
buffer.writeBytes(Packet.DELIMITER_BYTES); buffer.writeBytes(Packet.DELIMITER_BYTES);
buffer.writeBytes(packetBuffer); buffer.writeBytes(packetBuffer);
} finally {
packetBuffer.release(); packetBuffer.release();
} }
} }
} }
}
private byte toChar(int number) { private byte toChar(int number) {
return (byte) (number ^ 0x30); return (byte) (number ^ 0x30);

Loading…
Cancel
Save