Daniel Parker

There are definitely issues with cow strings, I'll quote from a ycombinator post, "In the early days of std::string multi-threading was much rarer, so reference counting was cheaper. As CPU...

My conclusion from the original build output is that the main issue has to do with your using a `std::basic_string` class that does not support stateful allocators. I believe the...

@barracuda156 The 32-bit warning that you reported earlier has been addressed on master. If you find any others, please report them, thanks.

Could you check again with the latest code on master? Thanks.

Oh sorry, the latest will still fail, this is a "comparison is always true due to limited range of data type" warning rather than a 32 bit issue per se....

Either use a pragma to selectively deactivate the warning in `basic_cbor_encoder`, or find some other way of expressing if (uint64_t(length)

Oh, good. We'll try the same strategy for msgpack.

When you get a chance, can you check master again?

Should be fixed on master

I think there's still a (innocuous) uninitialized warning in this one environment. I haven't seen it in any other. May need to suppress it.