Boost C++ Libraries: Ticket #2295: Inconsistent behavior when using 64 bit integer types

Sat, 06 Sep 2008 18:43:30 GMT

Note: tested on MS VC9 (without SP1)

anonymous — Fri, 28 Nov 2008 13:28:28 GMT

Update: the 2. problem seems to be caused by the MSVC streams bug, and the 1. might also probably caused by it.

Nevertheless, lexical_cast tests really should include tests for 64bit integer types.

nasonov — Thu, 04 Dec 2008 11:04:26 GMT

Nevertheless, lexical_cast tests really should include tests for 64bit integer types.

They do. Search for LCAST_TEST_LONGLONG in http://svn.boost.org/svn/boost/branches/release/libs/conversion/lexical_cast_test.cpp

Though, I don't understand/remember why it uses int64 rather than boost::long_long_type.

I should also add a test for boost::intmax_t type.

anonymous — Thu, 04 Dec 2008 19:11:29 GMT

Replying to nasonov:

Nevertheless, lexical_cast tests really should include tests for 64bit integer types.

They do. Search for LCAST_TEST_LONGLONG in http://svn.boost.org/svn/boost/branches/release/libs/conversion/lexical_cast_test.cpp

They do indeed. Didn't notice it before when I was looking.

Though, I don't understand/remember why it uses int64 rather than boost::long_long_type.

The link you posted tests boost::long_long_type if BOOST_HAS_LONG_LONG is defined (and seems fall back on int64 defending on few other defined).

I should also add a test for boost::intmax_t type.

May I suggest adding more tests that check conversion failure in the generic testing routines - there is currently not a single one failure test in them (and therefore not a single failure test for long long), there are only tests for around min, max, 0 and exp(10).

Eg. for "test_conversion_from_integral_to_string" something like my second use case would fit nicely: typedef std::basic_string<CharT> string_type; for(i = 0; i <= 10; ++i) {

string_type underflow_str = to_str<CharT>(limits::min()) + to_str<CharT>(i); BOOST_CHECK_THROW(lexical_cast<T>(underflow_str), bad_lexical_cast);

string_type overflow_str = to_str<CharT>(limits::max()) + to_str<CharT>(i); BOOST_CHECK_THROW(lexical_cast<T>(overflow_str), bad_lexical_cast);

} (this check overflow and underflow by appending digits, not by strictly increasing the min/max by +/-1, although that could also be done.)

status changed; resolution set

nasonov — Sun, 13 Sep 2009 14:48:29 GMT

status new → closed
resolution → fixed

(In [56170]) Fix #2295 (Inconsistent behavior when using 64 bit integer types).