Re: [Boost-users] [Review] Boost.Type Traits Extension by FredericBron

Subject: Re: [Boost-users] [Review] Boost.Type Traits Extension by FredericBron
From: John Maddock (boost.regex_at_[hidden])
Date: 2011-03-17 07:24:41

> * I note that is does not use an automated system like Doxygen of 
> producing
> the reference information.
> This seems to unfortunate, especially as there seem to be plans to change
> some names.
> I think we can be sure that there will be a fair number of glitches in the
> documentation when this happens :-(
>
> Although it is more work to start with (and does clutter the code -
> something that is made much less important by syntax coloring, so one can
> mentally ignore all the Doxygen comments)  it does make it much, much 
> easier
> to ensure that the code and the documentation of functions, classes... 
> stay
> in step.
>
> I would like to see the whole library Doxygen fully commented (ask if you
> think I can help) - replacing much of the current reference text.

Personally I wouldn't, sorry, but I'm still not completely sold on Doxygen 
:-(

> * I note that I didn't get the Boost or navigation icons or fonts from the
> index.html sandbox site.

That will be a non-issue when merged to Trunk.

> * The previous PDF version is fine (but of course lacking the new
> operators). A new PDF version is extremely desirable (ask if you think I 
> can
> help).

Also a non-issue once it's merged to Trunk, as the type_traits PDF gets 
built for each release.

> * Some simpler (trivial even) examples might help newcomers to the 
> library?
> The examples given are 'interesting' but quite complicated.
>
> * Floating point example(s) could refer to the Boost.Math library where
> type_traits are widely used?
>
> * A few items could be updated for the latest compilers like VC2010?
>
> has_nothrow_assign has no_throw_copy ...
>
> "Currently (May 2005) only Visual C++ 8 has the necessary compiler support
> to ensure that this trait "just works". "
>
> "Currently (May 2005) only Visual C++ 8 has the necessary compiler support
> to ensure that this trait "just works"."

Hehe, working on that now.  In any case this is the "old" type_traits, not 
the extension under review.

HTH, John.