On Thu, 25 Jul 2019 11:00:03 -0500, Scott Cheloha wrote:

> > However, if the "itp->it_value.tv_sec >= 0" is true, that means
> > that "itp->it_value.tv_sec" must also be true.
>
> which is not correct, as itp->it_value.tv_sec could be zero.

Sigh, still too early I guess.

> What we really want to check is:
>
> if (itp->it_value.tv_sec > 0 ||
>     (itp->it_value.tv_sec == 0 && itp->it_value.tv_usec > 0))
>
> which is nearly equivalent to the expansion above.  The expansion
> above ignores the sign of itp->it_value.tv_usec, so, not exactly the
> same.

Right.

 - todd

Reply via email to