Uploaded image for project: 'AMQ Broker'
  1. AMQ Broker
  2. ENTMQBR-5407

AMQPMessage.scanMessageData() creates unecessary buffer wrapper

XMLWordPrintable

      The AMQPMessage.scanMessageData(ReadableBuffer) method creates an unecessary buffer wrapper on every decode. The method has the equivalent of:

      decoder.setBuffer(data);
       try {
         //...use decoder..
       } finally {
         decoder.setByteBuffer(null);
       }
      

      Effectively it tries to use the thread-local decoder, then ensure the related buffer is discarded after.

      However the two setter uses are different, with the former taking the ReadableBuffer directly, and the latter method expected to be passed a ByteBuffer, which causes creation of a ReadableBuffer wrapper for it. By passing it null with that method, a needless wrapper object is created. One which is discarded on the next decode by that thread when it sets the next actual buffer. It should be using setBuffer(null).

              fnigro Francesco Nigro
              fnigro Francesco Nigro
              Roman Vais Roman Vais (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: