On 8/18/13, Christian Weisgerber <na...@mips.inka.de> wrote:
> patrick keshishian <sids...@boxsoft.com> 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                          na...@mips.inka.de

Reply via email to