Don't create intermediate string in msg2str

[indygreg]

msg2str is needlessly complicated. Use the std::string::assign has an overloaded instance that takes a char * and size_t. While you are there, consider changing the API to pass references instead of pointers. Or, at least check for NULL before doing anything.