Re: [libvirt] [PATCH 2/8] virtime: Resolve Coverity DEADCODE

2014-09-15 Thread Peter Krempa
On 09/13/14 15:27, John Ferlan wrote: > Coverity complains that because of how 'offset' is initialized to > 0 (zero), the resulting math and comparison on rem is pointless. > > According to the origin commit id '3ec128989', the code is a > replacement for gmtime(), but without the localtime() or G

[libvirt] [PATCH 2/8] virtime: Resolve Coverity DEADCODE

2014-09-13 Thread John Ferlan
Coverity complains that because of how 'offset' is initialized to 0 (zero), the resulting math and comparison on rem is pointless. According to the origin commit id '3ec128989', the code is a replacement for gmtime(), but without the localtime() or GMT calculations - so just remove this code and a