MsgPack::pack() and unpack() can not report pack/unpack error.

[hgoldfish]

if QVariantList contains some element that can not be packed, the packer just silently ignore this element and print warning, There is no way to report this error to upper caller.

QVariantList l;
l << 1 << 2 << objectCanNotPacked() << 3;
qDebug() << MsgPack::unpack(MsgPack::pack(l));

Then 1,2,3, is output.

[romixlab]

Sorry for late answer.
Precisely in this case I don't see any other choice than to throw an exception, but that would kill whole process if not caught...
Maybe add MsgPack::strictMode(true) which would enable exceptions in such cases, what do you think?

[hgoldfish]

I think, the MsgpackStream should be used to replace the MsgPack::pack() and MsgPack::unpack() using these operators:

MsgpackStream &MsgpackStream::operator<<(const QVariant &v);
MsgpackStream &MsgpackStream::operator>>(QVariant &v);

At the meantime, MsgpackStream can report errors via status() during serialization.

I implements a simple version based on your codes:

https://github.com/hgoldfish/qtnetworkng/blob/master/include/msgpack.h
https://github.com/hgoldfish/qtnetworkng/blob/master/src/msgpack.cpp