https://bugs.kde.org/show_bug.cgi?id=392768
Kevin Funk <kf...@kde.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |RESOLVED Latest Commit| |https://commits.kde.org/kio | |/27271405f4085bd8a398d41a75 | |ac52cb4f857efd Resolution|--- |FIXED --- Comment #15 from Kevin Funk <kf...@kde.org> --- Git commit 27271405f4085bd8a398d41a75ac52cb4f857efd by Kevin Funk, on behalf of Julien Goodwin. Committed on 09/01/2020 at 14:17. Pushed by kfunk into branch 'master'. SlaveBase::dispatchLoop: Fix timeout calculation Summary: Old version of the code: ms = qMax<int>(d->nextTimeout.elapsed() - d->nextTimeoutMsecs, 1); ... will mean the sleep is for as long as the timer has run *minus* the intended duration, so if nextTimeoutMsecs is ever set and the timer just started this becomes very negative, and 1ms is the result. Inverting the subtraction: ms = qMax<int>(d->nextTimeoutMsecs - d->nextTimeout.elapsed(), 1); Means sleeping for the remaining time, and so far my CPU seems much happier, with my KIO-HTTP using apps looking fine. Reviewers: chinmoyr, davidedmundson, dfaure, broulik Reviewed By: dfaure Subscribers: ngraham, kde-frameworks-devel Tags: #frameworks Differential Revision: https://phabricator.kde.org/D26364 M +1 -1 src/core/slavebase.cpp https://commits.kde.org/kio/27271405f4085bd8a398d41a75ac52cb4f857efd -- You are receiving this mail because: You are watching all bug changes.