fixed packExtendedTypeHeader for payload length = 32, 64, 128

Hi Taro,

For me your suggested change looks like micro optimization. I prefer 
readable, understandable code.
But if I should change my pull request, just give me a note.

Best regards,

Martin

Am 25.12.2014 um 09:42 schrieb Taro L. Saito:
>
> In msgpack-core/src/main/java/org/msgpack/core/MessagePacker.java 
> <https://github.com/msgpack/msgpack-java/pull/156#discussion-diff-22269817>:
>
> > @@ -442,18 +442,16 @@ public MessagePacker packValue(Value v) throws IOException {
> >
> >      public MessagePacker packExtendedTypeHeader(int extType, int payloadLen) throws IOException {
> >          if(payloadLen < (1 << 8)) {
> > -            if(payloadLen > 0 && (payloadLen & (payloadLen - 1)) == 0) { // check whether dataLen == 2^x
> > -                if(payloadLen == 1) {
> > -                    writeByteAndByte(FIXEXT1, (byte) extType);
> > -                } else if(payloadLen == 2){
> > -                    writeByteAndByte(FIXEXT2, (byte) extType);
> > -                } else if(payloadLen == 4) {
> > -                    writeByteAndByte(FIXEXT4, (byte) extType);
> > -                } else if(payloadLen == 8) {
> > -                    writeByteAndByte(FIXEXT8, (byte) extType);
> > -                } else {
>
> How about changing this as |else if (payloadLen == 16) { ... } else { 
> ... writeByteAndByte(EXT8, ... )}|?
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/msgpack/msgpack-java/pull/156/files#r22269817>.
>