----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5194/ -----------------------------------------------------------
Review request for Plasma. Summary ------- Some AbstractRunner subclasses do some set up in their match method which other threads need to wait on and it would be preferable if more threads for that runner were spawned until it was complete. See the kill runner for an example of this. In most cases, though possibly not all (but certainly in the case of the Kill runner), this is done to avoid blocking in the slot connected to by prepare(). Even if the prepare()-connect slot is run in a separate thread, however, there's still the need to mark the runner as not currently match-able, preventing unwanted thread spawning. This patch provides for this. It is complimentary and orthogonal to the idea of runners with re-entrant match() methods. Diffs ----- /trunk/KDE/kdelibs/plasma/abstractrunner.h 1169194 /trunk/KDE/kdelibs/plasma/abstractrunner.cpp 1169194 /trunk/KDE/kdelibs/plasma/private/abstractrunner_p.h 1169194 /trunk/KDE/kdelibs/plasma/runnermanager.h 1169194 /trunk/KDE/kdelibs/plasma/runnermanager.cpp 1169194 Diff: http://reviewboard.kde.org/r/5194/diff Testing ------- Tried with kill runner, works charmingly. Thanks, Aaron
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
