Re: [libvirt] [PATCH 6/9] virtime: Resolve Coverity DEADCODE

2014-09-12 Thread Peter Krempa
On 09/12/14 02:06, John Ferlan wrote: Coverity complains that because of how 'offset' is initialized to 0 (zero), the resulting math and comparison on rem is pointless. For the while (rem 0), the value of 'rem' must be between 0 and 86399 (SECS_PER_DAY = 86400ULL). Thus, the addition of

[libvirt] [PATCH 6/9] virtime: Resolve Coverity DEADCODE

2014-09-11 Thread John Ferlan
Coverity complains that because of how 'offset' is initialized to 0 (zero), the resulting math and comparison on rem is pointless. For the while (rem 0), the value of 'rem' must be between 0 and 86399 (SECS_PER_DAY = 86400ULL). Thus, the addition of offset (0) does nothing and the while (rem 0)