Opened 11 years ago
Closed 11 years ago
#5564 closed Patches (fixed)
lexical_cast performance boost for signed and unsigned char types
Reported by: | Owned by: | Antony Polukhin | |
---|---|---|---|
Milestone: | To Be Determined | Component: | lexical_cast |
Version: | Boost Development Trunk | Severity: | Optimization |
Keywords: | Cc: |
Description
We were looking through the sources of boost::lexical_cast library, and found a possible way of optimization.
According to the [27.6.2.5.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 [27.6.1.2.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
Attachments (1)
Change History (3)
by , 11 years ago
Attachment: | lexical_cast.diff added |
---|
comment:1 by , 11 years ago
Status: | new → assigned |
---|
Patch for the trunk version