Re: [boost] [timer] Boost Timer Library Version 2

Subject: Re: [boost] [timer] Boost Timer Library Version 2
From: Beman Dawes (bdawes_at_[hidden])
Date: 2011-09-30 12:06:23

On Tue, Sep 27, 2011 at 6:24 PM, Vicente J. Botet Escriba
<vicente.botet_at_[hidden]> wrote:
> Le 27/09/11 23:01, Vicente J. Botet Escriba a écrit :
>>
>> Le 27/09/11 15:54, Beman Dawes a écrit :
>>>
>>> While additional comments are always welcome, unless someone comes up
>>> with a showstopper I'm planning to move Version 2 to trunk and get on
>>> with other projects.
>>>
>>
> An additional remark,
>
> you are using the high_resolution_clock to measure the wall clock.
> high_resolution_clock is a system wide steady clock on platforms providing
> them, but system_clock (not steady) on the others. I guess that the user
> expects a steady clock while measuring the wall clock of the process.
> Unfortunately Boost.Chrono doesn't provides always a steady_clock class :(
>
> The documentation should warn the user of this possibility.

Good point!  Will do.

By the way, there was a late change to C++11 to add noexcept to the
chrono clock's now() function. Can the Boost.Chrono now() function
throw? The timer code assumes it doesn't.

--Beman