Details
-
Improvement
-
Status: Closed
-
Major
-
Resolution: Fixed
-
0.9
-
None
-
Patch Available
Description
Currently, TBinaryProtocol reads into a temporary buffer, then copies the results into the resulting std::string. For large strings (like a megabyte), the copy can be a performance issue.
The attached patch reads directly into the std::string instead.
Hmm... I think we should keep "prevent stack overflow for v. large strings"