Subject: Re: [boost] [Review] Formal Review of Proposed Boost.Chrono Library LAST DAY TODAY
From: Paul A. Bristow (pbristow_at_[hidden])
Date: 2010-11-15 11:06:07
> -----Original Message-----
> From: boost-bounces_at_[hidden] [mailto:boost-bounces_at_[hidden]]
> On Behalf Of Anthony Williams
> Sent: Monday, November 15, 2010 3:04 PM
> To: boost_at_[hidden]
> Subject: [boost] [Review] Formal Review of Proposed Boost.Chrono Library
> Boost.Chrono aims to implement the new time facilities in C++0x, as
> N2661 - A Foundation to Sleep On (http://www.open-
> - What is your evaluation of the design?
Seems carefully considered over some time.
Seems suitably open-ended with many possible time units and clocks -
No one clock suits all requirements, so wisely includes user clocks,
eBay clock, my_own_dodgy_clock, ...
Needs to provide examples showing how to use IO output (input?) Hr, Min, sec
in languages *other than English*.
I'm not sure the mechanism provided will work in practice?
Especially if using locales and expecting it to work with them?
This is such a basic need, that leaving it to the users to reinvent
repeatedly seems unhelpful.
More examples are always helpful. I didn't find all the ones referenced,
but this may be a filing problem using sandbox.
Did I miss examples of actually using Chrono with Boost.Units library?
> - What is your evaluation of the implementation?
> - What is your evaluation of the documentation?
Good - a few typos (sent separately). I would have used Doxygen reference
to reduce future maintenance of docs.
I'd like it *with Doxygen comments in the code*: without this, reference
docs can be hard to digest.
And as a user (including of my own code!), I also find it very useful to
have a standalone Doxygen.
Again the *with Doxygen comments in the code* makes it much more useful.
Adding Doxygen comments in the code is easiest when writing the code, but
perhaps a volunteer could add afterwards?
> - What is your evaluation of the potential usefulness of the library?
> - Did you try to use the library? With what compiler? Did you have any
Using MSVC 10 but without Boost.System library as included.
Puzzling compile problem that I didn't have time to investigate.
> - How much effort did you put into your evaluation? A glance?
A quick reading.
> - Are you knowledgeable about the problem domain?
Not yet - haven't had time ;-)
> And finally, every review should answer this question:
> - Do you think the library should be accepted as a Boost library?
--- Paul A. Bristow, Prizet Farmhouse, Kendal LA8 8AB UK +44 1539 561830 07714330204 pbristow_at_[hidden]
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk