Re: [Boost-users] [Chrono] How to report problems?

Subject: Re: [Boost-users] [Chrono] How to report problems?
From: Roland Bock (rbock_at_[hidden])
Date: 2010-05-27 10:08:37

Hi,

sorry for the late reply, did not see it among all the other stuff....

vicente.botet wrote:
> Hi,
> ----- Original Message ----- 
> From: "Roland Bock" <rbock_at_[hidden]>
> To: <boost-users_at_[hidden]>
> Sent: Thursday, April 15, 2010 10:48 AM
> Subject: [Boost-users] [Chrono] How to report problems?
> 
> 
>> Hi,
>>
>> after having received a push in the right direction by Howard yesterday,
> 
> Thanks Howard.
>  
>> everything works fine so far, except for for the following two details:
>>
>> a) I get some warnings about unused parameters when using the stopclock.
>> For instance:
>>
>> include/boost/chrono/stopwatch_reporter.hpp:81: warning: unused
>> parameter ‘ec’
>>
>> The fixes are trivial in all cases I encountered.
> 
> Yes I get it. I will check the warnings on the builds I can run. On which platform and with wich compiler are you working? Could you sent me a log with the warnings you found?

Ubuntu-8.04, 64bit, gcc-4.2.4, with the following warning flags:
-Wall -Wreorder -Wnon-virtual-dtor -Wno-non-template-friend
-Woverloaded-virtual -Wsign-promo -Wextra

Warnings are attached for the following line:

boost::chrono::stopclock<> stopclock;

>  
>> b) The thread_clock part of the tutorial looks pretty broken.
> 
> thread_clock was one of the last features I added and has been tested only on linux with gcc-4.2.1. I need to port it to Windows, but for the moment I have had no time. Could you give more details on how this is broken?
>  
>> Not sure if more things will come up, but if so, where should I send
>> them? Maybe this information would be a nice addition to the FAQ until
>> chrono makes it into boost?
> 
> You can send to this ML, but as the library is under developement maybe it is better to send you mails to the boost development list boost_at_list.boost.org.
>  
>> Thanks for this cool library!
> 
> Glad to hear it is useful for you.
> 
> Have you run the tests onyour platform? It would be great if you can make a pre-review of the library, this will surely help to improve the library before the formal review, could you?

We are using

boost::chrono::thread_clock
boost::chrono::thread_clock::time_point
boost::chrono::system_stopwatch

in production now (on an Ubuntu system). All numbers I checked so far
looked very reasonable :-)

I have no idea what is required for a pre-review. Is there some document
for that?

Regards,

Roland

> 
> Best,
> Vicente
> 
> 
> _______________________________________________
> Boost-users mailing list
> Boost-users_at_[hidden]
> http://lists.boost.org/mailman/listinfo.cgi/boost-users


/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopwatch_reporter.hpp: In instantiation of `boost::chrono::basic_stopwatch_reporter<Stopwatch, Formatter>::basic_stopwatch_reporter(boost::system::error_code&) [with Stopwatch = boost::chrono::stopwatch<boost::chrono::process_cpu_clock>, Formatter = boost::chrono::basic_time_formatter<char, std::char_traits<char>, std::allocator<char> >]':
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopclock.hpp:58:   instantiated from `boost::chrono::basic_stopclock<Clock, Formatter>::basic_stopclock(boost::system::error_code&) [with Clock = boost::chrono::process_cpu_clock, Formatter = boost::chrono::basic_time_formatter<char, std::char_traits<char>, std::allocator<char> >]'
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopclock.hpp:127:   instantiated from `boost::chrono::stopclock<Clock, Formatter>::stopclock(boost::system::error_code&) [with Clock = boost::chrono::process_cpu_clock, Formatter = boost::chrono::basic_time_formatter<char, std::char_traits<char>, std::allocator<char> >]'
/home/rbock/metafeed/sources/branches/ehe-2.4.beta/sources/BoostTests/src/Chrono.cpp:14:   instantiated from here
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopwatch_reporter.hpp:78: warning: unused parameter `ec'
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/scoped_suspend.hpp: In instantiation of `boost::chrono::scoped_suspend<Clock>::scoped_suspend(boost::system::error_code&) [with Clock = boost::chrono::process_cpu_clock]':
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopwatch_reporter.hpp:148:   instantiated from `void boost::chrono::basic_stopwatch_reporter<Stopwatch, Formatter>::report(boost::system::error_code&) [with Stopwatch = boost::chrono::stopwatch<boost::chrono::process_cpu_clock>, Formatter = boost::chrono::basic_time_formatter<char, std::char_traits<char>, std::allocator<char> >]'
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopwatch_reporter.hpp:120:   instantiated from `boost::chrono::basic_stopwatch_reporter<Stopwatch, Formatter>::~basic_stopwatch_reporter() [with Stopwatch = boost::chrono::stopwatch<boost::chrono::process_cpu_clock>, Formatter = boost::chrono::basic_time_formatter<char, std::char_traits<char>, std::allocator<char> >]'
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/stopclock.hpp:47:   instantiated from here
/home/rbock/metafeed_branches_ehe-2.4.beta_DEBUG/binaries/boost/1.41/include/boost/chrono/scoped_suspend.hpp:25: warning: unused parameter `ec'