D24962: [DeleteJob] Use a separate worker thread to run actual IO operation
David Faure
noreply at phabricator.kde.org
Sun Oct 27 22:00:30 GMT 2019
dfaure added a comment.
Any reason why you didn't implement my suggestion of
DeleteJobIOWorker *ioworker() {
if (!m_ioworker) {
...
}
return m_ioworker;
}
[...]
QMetaObject::invokeMethod(ioworker(), "rmfile", [...]);
?
A call to an initSomething() method can easily be forgotten, while an on-demand getter ensure that the worker is created when it's needed (for the first time).
Sorry for the nitpicking :-)
REPOSITORY
R241 KIO
REVISION DETAIL
https://phabricator.kde.org/D24962
To: meven, dfaure
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191027/2fb0f00c/attachment.html>
More information about the Kde-frameworks-devel
mailing list