<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/104802/">http://git.reviewboard.kde.org/r/104802/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs.</div>
<div>By Dawit Alemayehu.</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch attempts to mitigate the unintended crashes that might result from using QDialog::exec in kdelibs. Since nested event loops are potential sources of inadvertent crashes, this patch attempts to prevent that by changing how dialogs are created in kdelibs. All blocking dialog calls, i.e. those that invoke QDialog.exec(), are wrapped with QPointer and the QPointer is checked once QDialog.exec returns. See http://www.kdedevelopers.org/node/3919 for more details.
Note that I am aware of other classes that create nested event loops (e.g. QProcess), but this fix is only applicable to QDialog usage.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kdeui/colors/kcolordialog.cpp <span style="color: grey">(95bb7f5)</span></li>
<li>kdeui/dialogs/kedittoolbar.cpp <span style="color: grey">(bb80952)</span></li>
<li>kdeui/dialogs/kinputdialog.cpp <span style="color: grey">(2801c00)</span></li>
<li>kdeui/dialogs/kpixmapregionselectordialog.cpp <span style="color: grey">(11d964b)</span></li>
<li>kdeui/dialogs/kshortcutsdialog.cpp <span style="color: grey">(a73f8f2)</span></li>
<li>kdeui/dialogs/kshortcutseditor.cpp <span style="color: grey">(5984a9d)</span></li>
<li>kdeui/findreplace/kfinddialog.cpp <span style="color: grey">(de2dd90)</span></li>
<li>kdeui/fonts/kfontdialog.cpp <span style="color: grey">(9bea490)</span></li>
<li>kdeui/widgets/ktextedit.cpp <span style="color: grey">(1e58706)</span></li>
<li>kdeui/xmlgui/kmenumenuhandler_p.cpp <span style="color: grey">(d8c82b6)</span></li>
<li>kfile/kdiroperator.cpp <span style="color: grey">(18ffc34)</span></li>
<li>kfile/kdirselectdialog.cpp <span style="color: grey">(e0dcafa)</span></li>
<li>kfile/kfileplaceeditdialog.cpp <span style="color: grey">(5537551)</span></li>
<li>kio/kfile/kacleditwidget.cpp <span style="color: grey">(d89429f)</span></li>
<li>kio/kfile/kencodingfiledialog.cpp <span style="color: grey">(4686065)</span></li>
<li>kio/kfile/kfiledialog.cpp <span style="color: grey">(d121e4d)</span></li>
<li>kio/kfile/kicondialog.cpp <span style="color: grey">(b7d646f)</span></li>
<li>kio/kfile/kpropertiesdialog.cpp <span style="color: grey">(feb0c9e)</span></li>
<li>kio/kfile/kurlrequesterdialog.cpp <span style="color: grey">(8ee29e1)</span></li>
<li>kio/kio/jobuidelegate.cpp <span style="color: grey">(85679c2)</span></li>
<li>kio/kio/kbuildsycocaprogressdialog.cpp <span style="color: grey">(fba30ec)</span></li>
<li>kio/kio/passworddialog.cpp <span style="color: grey">(faf0c77)</span></li>
<li>kio/kio/paste.cpp <span style="color: grey">(ca451fb)</span></li>
<li>kio/kssl/kcm/cacertificatespage.cpp <span style="color: grey">(0a269a3)</span></li>
<li>knewstuff/knewstuff2/ui/downloaddialog.cpp <span style="color: grey">(b4d2dcd)</span></li>
<li>knewstuff/knewstuff2/ui/kdxsbutton.cpp <span style="color: grey">(e8f8c83)</span></li>
<li>knewstuff/knewstuff3/knewstuffbutton.cpp <span style="color: grey">(9c14e99)</span></li>
<li>kparts/browserrun.cpp <span style="color: grey">(c89829d)</span></li>
<li>kutils/kpluginselector.cpp <span style="color: grey">(505e53f)</span></li>
<li>nepomuk/ui/tagwidget.cpp <span style="color: grey">(7c59922)</span></li>
<li>nepomuk/utils/searchwidget.cpp <span style="color: grey">(f46e72a)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104802/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>