Review Request 113535: Fix build with latest ThreadWeaver

Christoph Feck christoph at maxiom.de
Fri Nov 1 18:10:01 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113535/
-----------------------------------------------------------

(Updated Nov. 1, 2013, 6:10 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Plasma.


Repository: plasma-framework


Description
-------

With recent API updates in ThreadWeaver, "enqueueRaw()" was obsoleted. According to ThreadWeaver maintainers, the correct way to restart threads is to call "reschedule()", but I am not sure if my fix is correct, partly because the comment seems to be in the wrong place.


Diffs
-----

  src/plasma/private/runnerjobs_p.h dfc2aca 
  src/plasma/runnermanager.cpp ee4851f 

Diff: http://git.reviewboard.kde.org/r/113535/diff/


Testing
-------

Compiles, no further testing.


Thanks,

Christoph Feck

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20131101/ac349629/attachment-0001.html>


More information about the Plasma-devel mailing list