<html>

 <body>

  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">

   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">

    <tr>

     <td>

      This is an automatically generated e-mail. To reply, visit:

      <a href="https://git.reviewboard.kde.org/r/127441/">https://git.reviewboard.kde.org/r/127441/</a>

     </td>

    </tr>

   </table>

   <br />

<p>

Ship it!

</p>

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Nice work, please commit to the Applications/16.04 branch and then merge to master.</p></pre>

 <br />

<p>- Andreas Cord-Landwehr</p>

<br />

<p>On März 22nd, 2016, 8:19 nachm. UTC, Hartmut Riesenbeck wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">

 <tr>

  <td>

<div>Review request for KDE Edu.</div>

<div>By Hartmut Riesenbeck.</div>

<p style="color: grey;"><i>Updated März 22, 2016, 8:19 nachm.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

parley

</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">When the app is quited app.exec() doesn't return. The process stays in

sleeping state. Removed app.setQuitOnLastWindowClosed(false) because

the quit() or exec() method of the application were not invoked

somewhere else in the application.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">After this the app ends in the correct way. But after quit is invoked,

there are sporadic segmentation faults and the following message was

shown on stderr:

    QCoreApplication::postEvent: Unexpected null receiver

This was fixed by removing zero pointer access in destuctor of

ParleyDocument.</p>

<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Added missing this pointer on KEduVocDocument allocation.</p></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>src/main.cpp <span style="color: grey">(2e87baa93f81349a187d620be2468f8a1bb4aca1)</span></li>

 <li>src/parleydocument.cpp <span style="color: grey">(cf4c81c3f8f35a99389d61a75d934cff3170b960)</span></li>

 <li>src/parleymainwindow.cpp <span style="color: grey">(d6f86debd32c1e77bbcca86a160906674960dce9)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>