<table><tr><td style="">dkurz updated the summary for this revision.<br />dkurz updated the test plan for this revision.<br />dkurz updated this revision to Diff 6344.<br />dkurz added a comment.</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D2602" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>With raw pointers, we have no way of telling if an object on the heap was deleted by its parent. On the other hand, QPointers are reset to null, which is easily detectable. The Q_ASSERTs make sure that everything was deleted as intended, and they would not work with raw pointers, because they would definitely still point to the location where the deleted objects were once created.</p>

<p>But I can see that you are still not convinced, which is fine. Even if I had convinced you, the Q_ASSERT patch should probably have been a patch of its own anyway.</p>

<p>The new patch only fixes the memory problems. When creating it, I realized  that the old patch did only fix one of them, because I uploaded an old version, sorry for that.</p>

<p>I updated the summary accordingly. If the patch is OK now, please commit it for me, since I do not have commit access.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKDEPIM KDE PIM</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D2602?vs=6318&id=6344" rel="noreferrer">https://phabricator.kde.org/D2602?vs=6318&id=6344</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2602" rel="noreferrer">https://phabricator.kde.org/D2602</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>kmail/src/kmmainwidget.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>dkurz, KDE PIM<br /><strong>Cc: </strong>mlaurent, kde-pim, spencerb, dvasin, winterz, smartins, vkrause, knauss, dvratil<br /></div>