[boost] lexical_cast update + Boost.Test question

From: Terje Slettebø (tslettebo_at_[hidden])
Date: 2005-06-16 14:11:04

Hi.

lexical_cast has now been updated in the CVS, with the following, as has
been reported as bugs or suggested improvements:

- Fixed so it doesn't trim any whitespace when converting (it used to throw
on " 123" -> int, but not on "123 " -> int, which was inconsistent).
- Fixed reversed bad_lexical_cast constructor arguments.
- lexical_cast now uses pass-by-const reference.

For compilers lacking support for partial specialisation of class templates
(like MSVC 6), it falls back on pass-by-value.

Sorry for the late update. It's been quite busy at work lately, so it's been
hard to concentrate on non-work projects, but I'm now more on top of things.
I'll read up on any lexical_cast-related postings, and can make subsequent
updates, should it be desired. I'll also check up the results after the next
regression test run, to see if there are any remaining issues.

The changes are of course done in cooperation with Kevlin.

It has been tested on Intel C++ 7.1, MSVC 2005 Final Beta, g++ 3.4 and
Borland C++ Builder X, and all tests pass there. I get errors (from
Boost.Test) when trying to run the test on MSVC 6, but testing lexical_cast
by itself seems to work. Might Boost.Test have a problem with MSVC 6?

By the way, I don't see the changes using the Boost ViewCVS, but it's listed
e.g. with "log". Are the ViewCVS results cached, somehow, or may the commit
have ended up on some other place than the trunk? I did make sure to check
out the trunk, before commit.

Regards,

Terje