[PATCH] Add errorList() and errorListWId() to KMessageBox

Matt Rogers matt.rogers at kdemail.net
Sat Oct 2 04:50:33 BST 2004


On Friday 01 October 2004 09:10 pm, Michael Pyne wrote:
> Hi all,
>
> Attached is a patch to kdelibs/kdeui/kmessagebox.{h,cpp} that adds support
> for the errorList() and errorListWId() convienience functions to
> KMessageBox.  I would like this support for JuK, as there are a few code
> paths that process a bunch of files sequentially, and each file may have an
> error.
>
> This is currently handled by either reporting the errors in separate
> message boxes (which is what I'm fixing right now, imagine 50 error message
> boxes in a row!), coercing a QStringList into a QString joined by newlines
> (this can make the end dialog needlessly tall), or by using a different
> dialog that does provide a list (which loses the "Error" property).
>
> The functions are all static, so there should be no problem with binary
> compatibility.
>
> Please let me know if it is alright for me to commit this.
>
> Regards,
>  - Michael Pyne

The patch looks good to me.

Matt




More information about the kde-core-devel mailing list