2
0
mirror of https://github.com/boostorg/convert.git synced 2026-01-19 04:02:18 +00:00
This commit is contained in:
Vladimir Batov
2014-07-03 15:34:26 +10:00
parent ab989f4440
commit 954eeb01b5

View File

@@ -9,36 +9,4 @@ in the reference section. Users will find this helpful too.
* The reference section could be much improved by adding brief Doxygen comments to
the files, functions etc.
* In my opinion the documentation should start by explaining how to convert between strings and numeric values.
* Next it could tell how to deal with failed conversions.
* Finally it could explain how to deal with stringstream modifiers.
* The documentation doesn't seem to say anything about conversion to strings.
* higher density of comments if any other authors ever need to work on the library
* Some comments of the //-comment-type begin on the zero'th column, whereas some begin at indented locations. That is not clean.
* There is a rather high number of comments of the type TBD. These are good for noting future goals, but also lend an uneasy feeling about the stability of the code.
* It might be useful to establish the range of application and the differences between convert and lexical_cast.
Jeroen
I'd like an overload without the Converter const & as well, defaulting to a converter chosen via a customisation point.
Roland
I think it would be relatively simple to get to a much leaner interface.
For instance,
* convert could have a converter template parameter, defaulting to the
stringstream converter
* convert::from() could have an overload that just takes the "from"
parameter and provides the default-constructed converter itself.
* convert could have a conversion operator for the OutType, which does
the same as convert::value()