[KDE/Mac] Review Request 120800: prevent starting a QTimer with a negative interval

[René J.V. Bertin]

> On Oct. 27, 2014, 10:17 p.m., David Faure wrote:
> > Thanks for looking into this, runtime warnings bother me too. But is there any point in that debug statement? Debugging? I'm curious too about how the timer could still be active at the time of the pausing, if it was after interval(). Maybe this is because setInterval was called later. But that means testing  mStarted.secsTo( mPaused ) at the time of the call to pause() too? Anyway, better finish the debugging (to fully understand the issue) than commit debug output and hope someone else will finish the debugging :-)

> I'm curious too about how the timer could still be active at the time of the pausing

Hard to be completely sure (that's why I had a copious debug output) but it must be related to the suspension. If I understand the code correctly, this function is called often enough that the remainder simply will never go negative. I don't know if kmail/kontact go offline on Linux when suspending the system, but this clearly isn't the case on OS X, allowing mStarted.secsTo to go off the charts.


- René J.V.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/120800/#review69220
-----------------------------------------------------------


On Oct. 27, 2014, 11:21 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/120800/
> -----------------------------------------------------------
> 
> (Updated Oct. 27, 2014, 11:21 p.m.)
> 
> 
> Review request for Akonadi and KDE Software on Mac OS X.
> 
> 
> Repository: akonadi
> 
> 
> Description
> -------
> 
> After waking my Mac from (a night's) sleep, I noticed a quickly repeating error message about starting a QTimer with a negative interval. I could trace this to the PausableTimer instance in collectionscheduler.cpp .
> The attached patch prevents the error condition by clipping the calculated remaining time interval to 0 .
> I've left a debug statement that can of course doesn't need to be committed.
> 
> 
> Diffs
> -----
> 
>   server/src/collectionscheduler.cpp 8d4cd5c 
> 
> Diff: https://git.reviewboard.kde.org/r/120800/diff/
> 
> 
> Testing
> -------
> 
> On OS X 10.6.8 with kdelibs 4.14.2 and kdepim* 4.13.3
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20141027/d1eb3660/attachment.html>