<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/107635/">http://git.reviewboard.kde.org/r/107635/</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;">Do you know if it is kdelibs' intention not to delete this kind of object? Valgrind complains the pointer is still reachable with this change, and it's why it was introduced in the first place.</pre>

 <br />

<p>- Raphael</p>

<br />

<p>On April 10th, 2013, 10:55 p.m. UTC, Sandro Mani wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">

 <tr>

  <td>

<div>Review request for KDE Utils and Raphael Kubo da Costa.</div>

<div>By Sandro Mani.</div>

<p style="color: grey;"><i>Updated April 10, 2013, 10:55 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;">Don't delete the KPart in the MainWindow destructor

Details:

This fixes the warning

KXMLGUIClient::~KXMLGUIClient: 0x1c77eb8 deleted without having been removed from the factory first. This will leak standalone popupmenus and could lead to crashes.

which appears when closing ark.

See also https://bugs.kde.org/show_bug.cgi?id=261538#c32</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>app/mainwindow.cpp <span style="color: grey">(59efe78515f6f6b2e0cb7bba4b4794c2cf86f846)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>