Peter M. Groen / FlashMQ

Shifting signed integers is undefined and only sometimes produced
unexpected results.

Was detected in packet identifiers in QoS testing.

Instead of getting it from the sender of a packet. Sometimes there was
no sender.

This fixes a crash on retained messages, because those newly created
packets didn't have a sender to get the threaddata from. So, using a
special object for it is easier and more robust.

I kind of need a config file parser first, so I'm going to make that.

The MQTT docs showed a wrong condition for checking malformed packets.

And, we have to grow buffers to MAX_PACKET_SIZE, otherwise we can't
process. I still have some inteligent buffer shrink logic in mind.

There still is a bug: writing a very big packet. I wrote the test case
already.