mozilla-services/powerhose — Issue #11: Don't create intermediate string in msg2str
Author: indygreg
Created: 2012-03-01 23:12:40+00:00
State: open

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.