On 8/18/13, Christian Weisgerber <[email protected]> wrote:
> patrick keshishian <[email protected]> wrote:
>
>> unsquandering ... (hopefully)
>
> I think most of this is pointless and misleading. The timeout value
> that is passed around is NOT a time_t. It's a time offset in the
> range 0 .. 3600*1000 milliseconds. Putting it into a time_t is an
> abuse of that type.
I agree. The purpose of nextTimeout(), seems to be, to returning
an interval in milliseconds. That is then passed to Timer::setTimeout().
--patrick
>> +--- lib/Timer.hh.orig Fri Mar 18 01:07:09 2005
>> ++++ lib/Timer.hh Sat Aug 17 22:43:03 2013
>> +@@ -37,16 +37,12 @@ struct timeval;
>> + namespace bt {
>> +
>> + // use a wrapper class to avoid the header as well
>> +- struct timeval {
>> +- long tv_sec;
>> +- long tv_usec;
>> ++ struct timeval : public ::timeval {
>
> Now THIS is a good catch.
>
> --
> Christian "naddy" Weisgerber [email protected]
