chinmoyr added inline comments.


> davidedmundson wrote in slaveinterface.cpp:106
> That wasn't there in the old code because it has this guard:
> const qint64 diff = currentTime - d->start_time;
> (effectively calculating elapsed)
> Then
> if (diff - d->last_time >= 900) {
> So we known first-last is non-zero
> Should the new code be
> ?
> It seemed to work for me.

Indeed that should be the code here.
The old code checked the elapsed time since last "calcSpeed()" call whereas the 
new code checks the elapsed time since the timer started.

> broulik wrote in slaveinterface.cpp:113
> We likely get a division by zero here for some reason

I doubt it will be zero here. `elapsed_time` stores elapsed time since the 
timer started so it will continue to grow (?) and the difference will always be 
> 0.

  R241 KIO


To: chinmoyr, dfaure
Cc: davidedmundson, broulik, bruns, kde-frameworks-devel, michaelh, ngraham

Reply via email to