<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/105920/">http://git.reviewboard.kde.org/r/105920/</a>
     </td>
    </tr>
   </table>
   <br />





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The reason it was not applied is probably because I forgot to submit it :) Thanks for taking care of doing it.

If I am not mistaken the reason KModifierKeyInfoProvider inherits from QWidget is historical: it used to be necessary to inherit from QWidget to be able to setup an XEvent filter, but it is no longer the case.</pre>
 <br />







<p>- Aurélien</p>


<br />
<p>On August 7th, 2012, 9:21 p.m., Frank Reininghaus wrote:</p>






<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, Michael Leupold and Aurélien Gâteau.</div>
<div>By Frank Reininghaus.</div>


<p style="color: grey;"><i>Updated Aug. 7, 2012, 9:21 p.m.</i></p>






<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;">The base class of KModifierKeyInfoProvider is QWidget although it seems that this class does not really need anything that QWidget provides and QObject doesn't.

This seems to be related to Dolphin crashes that can happen with some styles and some particular settings, see also https://bugs.kde.org/show_bug.cgi?id=264923.

According to https://bugs.kde.org/show_bug.cgi?id=264923#c11, Aurelien wrote the patch which I'm uploading here. Is there any information on why KModifierKeyInfoProvider was derived from QWidget in the first place, or why Aurelien's patch never made it into kdelibs?

The root cause of the crash is probably something else, but having a class inherit QWidget rather than QObject when it's not actually needed looks strange to me in any case. Unless there's a good reason that I'm overlooking here, of course - in that case, I'm curious what it is :-)</pre>
  </td>
 </tr>
</table>




<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=303169">303169</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>kdeui/util/kmodifierkeyinfoprovider_p.h <span style="color: grey">(2df79bc)</span></li>

 <li>kdeui/util/kmodifierkeyinfoprovider_x11.cpp <span style="color: grey">(c6477f9)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/105920/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>