https://bugs.kde.org/show_bug.cgi?id=389611

Fabian Vogt <fab...@ritter-vogt.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|REPORTED                    |RESOLVED
      Latest Commit|                            |https://commits.kde.org/mil
                   |                            |ou/c918c2840bcbbc52da72f163
                   |                            |c8494db227db3600

--- Comment #2 from Fabian Vogt <fab...@ritter-vogt.de> ---
Git commit c918c2840bcbbc52da72f163c8494db227db3600 by Fabian Vogt.
Committed on 18/06/2019 at 16:55.
Pushed by fvogt into branch 'Plasma/5.12'.

Don't give up if no results arrive after 500ms

Summary:
Currently it resets the entire context if no results appear after 500ms,
which means no new results can appear. This makes no sense.

RunnerManager does not emit any matchChanged signal while the runners update
the matches too quickly. The way this delay is implemented can result in never
actually emitting the signal until all runners are done though.
Thus it's actually likely to hit this situation.

Test Plan:
Test results appear now, even if the first batch of runners is slow.
List of matches still clears if nothing was found.

Reviewers: #plasma

Subscribers: broulik, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D21605

M  +6    -1    lib/sourcesmodel.cpp
M  +3    -0    lib/sourcesmodel.h

https://commits.kde.org/milou/c918c2840bcbbc52da72f163c8494db227db3600

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to