Re: [boost] [chrono] review version

Subject: Re: [boost] [chrono] review version
From: Paul A. Bristow (pbristow_at_[hidden])
Date: 2010-11-02 05:59:14

Excellent - I hope we can get this reviewed and into a release Real Soon
Now.

Paul

PS Some nits

I think you are missing an @ below:

Appendix B: Rationale
See [http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2008/n2661.htm N2661
- A Foundation to Sleep On]

And I think Andrew? got a poor thanks for his efforts - or they we e not
enough ;-)

"Thanks to Adrew Chinoff for his help polishing the documentation."

3. __high_precisionclock_ is missing a _?

constexpr don't used for min(), max() and zero()  - 'don't'  should be
'isn't'?

> -----Original Message-----
> From: boost-bounces_at_[hidden] [mailto:boost-bounces_at_[hidden]]
> On Behalf Of vicente.botet
> Sent: Monday, November 01, 2010 11:28 PM
> To: boost_at_[hidden]
> Cc: anthony_at_[hidden]
> Subject: [boost] [chrono] review version
> 
> Hi,
> 
> I have made some modifications to Boost.Chrono before the review (Version
> 0.7.0).
> 
> The library can be downloaded from
> http://www.boostpro.com/vault/index.php?action=downloadfile&filename=chr
> ono.zip&directory=System& and the documentation is online on the sandbox
>
http://svn.boost.org/svn/boost/sandbox/chrono/libs/chrono/doc/html/index.ht
> ml or in PDF format
> http://svn.boost.org/svn/boost/sandbox/chrono/libs/chrono/doc/chrono.pdf
> 
> 
> Boost.Chrono depends on Boost.Ratio
> http://www.boostpro.com/vault/index.php?action=downloadfile&filename=rati
> o.zip&directory=Math%20-
> %20Numerics&PHPSESSID=2d7859f3e812c993f5a9e2d9900dfee8 and optionaly
> Boost.System Header Only
> http://www.boostpro.com/vault/index.php?action=downloadfile&filename=sys
> tem.zip&directory=System&PHPSESSID=2d7859f3e812c993f5a9e2d9900dfee8.
> 
> Next follows the major changes:
> 
> [*Features:]
> 
> * Boost_Chrono is now a configurable header only lib that allows the user
to
> choose in addition if the windows file is included or not.
> * Added clock_string<> traits.
> * Define chrono_io for all the clocks
> * Add input of process_times representation
> 
> [*Implementation:]
> 
> * Moved some details to static_integer directory.
> * Use of detail/win files to avoid the use of windows.h file.
> * Completed the error_code handling.
> * Works now with BOOST_SYSTEM_NO_DEPRECATED.
> 
> [*Fixes:]
> 
> * Fix some warnings.
> * Fix original error on Mac
> * Don't fix the link with boost_system to static.
> 
> [*Test:]
> 
> * Added test on process and thread clocks.
> * Moved to lightweight_test.hpp.
> * Able to test multiple configurations.
> 
> [*Doc:]
> 
> * Removed some not useful parts as the history, the test and the tickets.
> 
> Best,
> _____________________
> Vicente Juan Botet Escribá
> http://viboes.blogspot.com/
> _______________________________________________
> Unsubscribe & other changes:
http://lists.boost.org/mailman/listinfo.cgi/boost