Re: Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-24 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/#review36469 --- This review has been submitted with commit 20b9d17e4c594765c19

Re: Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-24 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/ --- (Updated July 25, 2013, 4:57 a.m.) Status -- This change has been mar

Re: Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-24 Thread Aaron J. Seigo
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/#review36454 --- Ship it! Ship It! - Aaron J. Seigo On July 23, 2013, 11:23

Re: Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-24 Thread Kevin Ottens
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/#review36430 --- Looks good overall, I still think a typedef for QSharedPointer

Re: Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-23 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/ --- (Updated July 23, 2013, 1:23 p.m.) Review request for KDE Frameworks and P

Re: Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-23 Thread Kevin Ottens
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/#review36354 --- Probably worth adding a typedef for QSharedPointer, just like

Review Request 111650: Adjust RunnerManager to new ThreadWeaver API using JobPointer

2013-07-23 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/111650/ --- Review request for KDE Frameworks and Plasma. Description --- ThreadW