[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: branches/KDE/4.4/kdelibs/kio/kio
From:       David Faure <faure () kde ! org>
Date:       2010-02-04 19:02:30
Message-ID: 201002042002.30632.faure () kde ! org
[Download RAW message or body]

On Thursday 04 February 2010, Dawit Alemayehu wrote:
> +    /*
> +      If startTimerDelayed is true, we still have jobs could not be
> scheduled +      due to limit enforcements. In order to avoid high CPU
> usage, however, we +      start the slaveTimer after 1 sec. Otherwise, the
> above while loop will be +      run in a tight loop and will peg the CPU.
> See BR# 224857... +    */
> +    if (startTimerDelayed) {
> +       kDebug(7006) << "Rescheduling remaining jobs immediately...";
> +       startTimerDelayed = false;
> +       slaveTimer.start(0);
> +    }

The comment says "after 1 sec" but the code says start(0); what do I miss?

-- 
David Faure, faure@kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. Konqueror (http://www.konqueror.org).
[prev in list] [next in list] [prev in thread] [next in thread]