[PATCH] thread-safe queued KDialogs
Thiago Macieira
thiago at kde.org
Sat Nov 22 22:08:00 GMT 2008
Sebastian TrĂ¼g wrote:
>The attached patch makes queued dialogs thread-safe. With it, queued
>KMessagesBoxes and thus, the default KMessage messasge handler.
>This fixes a crash in KRunner when KMimeType tried to complain about
> missing mimetypes using KMessage which in turn uses KMessageBox which
> tries to open a QWidget from a thread which is not the GUI thread which
> triggers an assert. :)
>
>May I apply?
What's the number 20? Where did the 20 milliseconds timer come from? Why
not 25?
I know it's not your code, but maybe someone else knows where this value
came from. Let's not keep arbitrary things.
--
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
PGP/GPG: 0x6EF45358; fingerprint:
E067 918B B660 DBD1 105C 966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20081122/cd4f6d91/attachment.sig>
More information about the kde-core-devel
mailing list