Ticket #5564 (closed Patches: fixed)
lexical_cast performance boost for signed and unsigned char types
|Reported by:||Bolotov Ilya, Mishakov Daniil <daniil987@…>||Owned by:||apolukhin|
|Milestone:||To Be Determined||Component:||lexical_cast|
|Version:||Boost Development Trunk||Severity:||Optimization|
We were looking through the sources of boost::lexical_cast library, and found a possible way of optimization.
According to the [188.8.131.52.4 Character inserter function templates] of Programming languages - C++ standard, basic_ostream::operator<< reinterpret_casts unsigned char* and signed char* to char* and calls operator<<. So we can convert from unsigned char* and signed char* types to char* type and use optimized boost::lexical_cast coversion methods without construction of basic_streambuf<CharT> objects. Same approach can be used with signed char and unsigned char (we can just static_cast them to char type).
According to the [184.108.40.206.3 basic_istream::operator>>] of Programming languages - C++ standard, basic_istream::operator>> for unsigned chars and signed chars behaves the same way, as for char type. boost::lexical_cast library treats remained in stream data as an error, so when the sizeof(CharT) is bigger than sizeof(char), the there will remain some data in stream and such situation can be interpreted as error. We can detect such cases at compile time, using BOOST_STATIC_ASSERT( sizeof(CharT) == sizeof(signed char) );
Using those optimizations, we won't construct basic_streambuf<CharT> object, so there will be no dynamic memory allocation and we will use the lexical_cast algorithms optimized for char and char* types.
Patch contains the required modifications and was tested on gcc4.5