On Friday, June 27, 2003, at 8:59 AM, Edward Diener wrote:

[EMAIL PROTECTED] wrote:
From: Edward Diener [mailto:[EMAIL PROTECTED]
[SNIP]
"The Tokenizer documentation for char_separator tokenizer
function states that the default argument for the second template type is "char_traits<char>". This is incorrect. The source code in token_functions.hpp clearly shows that the default argument is "std::basic_string<Char>::traits_type". Could this please be corrected ?"

I don't think that the documentation should change (not counting the change from char to Char), but rather the implementation.
std::basic_string<Char>::traits_type will always be
std::char_traits<Char>. If there are no objections to this (there
might have been some compiler issue that justified the code, although there's no such indication in the CVS history), I'll change token_functions.hpp accordingly.

You will probably want to change escaped_list_separator in the same way.

Since


std::basic_string<Ch>::traits_type == std::char_traits<Ch>

is always true, shouldn't both the code and documentation be changed? There is no need to hide the class's identity.

Daryle

_______________________________________________
Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost

Reply via email to