Thrift: Revamp TMemoryBuffer constructors.
authorDavid Reiss <dreiss@apache.org>
Mon, 4 Feb 2008 21:56:27 +0000 (21:56 +0000)
committerDavid Reiss <dreiss@apache.org>
Mon, 4 Feb 2008 21:56:27 +0000 (21:56 +0000)
commitf7baf549bcea8793f7882eb7e01cb3affaea97b8
treeb09ef9b309f15139699de5a417612121033d97bb
parentadb5cad7d32c1dd2f11b18b8d0f14b6955df3606
Thrift: Revamp TMemoryBuffer constructors.

Summary:
There were some weird cases where the implicit conversion from
const char* to std::string was causing the wrong constructor to be called.
There wasn't really a clean workaround, so we're dropping the string
constructors.

Reviewed By: mcslee

Test Plan:
Ran the test.
Grepped around the /projects tree for uses that had to fixed,
and fixed them.

Revert Plan: ok

Other Notes:
This risk was pointed out by Ben Maurer.

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@665461 13f79535-47bb-0310-9956-ffa450edef68
lib/cpp/src/transport/TTransportUtils.h
test/TMemoryBufferTest.cpp