Implement queueing directly in KDialogJobUiDelegate
Review Request #113516 - Created Oct. 31, 2013 and submitted
Implement queueing directly in KDialogJobUiDelegate by queueing calls to KDialogJobUiDelegate::Private::next which shows 1 KMessageBox at the time.
Generally looks good, but see the comments below. Also, the apidocs should state that messageboxes for any given job will be queued up, but that queueing will not happen across jobs (or globally); I believe this is a change in behaviour from kdelibs4, where the queueing was application-global.
Would it not be marginally more efficient (and still correct) to put this test at the end of the method, with the invokeMethod in an else clause?
This will be application model, I believe. Might be worth mentioning the class apidocs (and/or providing a way to make it window-modal instead).
For the sake of being "obviously correct", this should probably be enqueue (since you use dequeue to remove).