<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://svn.reviewboard.kde.org/r/5549/">http://svn.reviewboard.kde.org/r/5549/</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;">Nice catch, but it cannot be fixed at that place. If you are setting the same proxy style to multiple widgets, deleting the first of them would delete the proxy style. So it should be decided per subclass what to do.

The proper fix is to apply your patch inside KLineEditStyle (klineedit_p.h:189). This is also how the KHTMLProxyStyle does it.</pre>

 <br />

<p>- Christoph</p>

<br />

<p>On October 8th, 2010, 10:07 a.m., Boudewijn Rempt wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 Boudewijn Rempt.</div>

<p style="color: grey;"><i>Updated 2010-10-08 10:07:23</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;">For instance in KLineEdit::init(), we have a line like:

d->style = new KLineEditStyle(this);

This gives the impression that KLineEditStyle has QObject-based parent-child relation and will be deleted automatically. That wasn't true, because QStyle while being a QObject doesn't have a QObject* parent paramater in its constructor. So this patch sets the parent explicitly in KdeUiProxyStyle </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>trunk/KDE/kdelibs/kdeui/widgets/kdeuiwidgetsproxystyle.cpp <span style="color: grey">(1182756)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>