<table><tr><td style="">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/D7291" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Objects that live on the stack should not be subject to heap memory management; something that definitively hasn't changed. Memory management in Qt is already quite confusing. This change supports local reasoning about the code. This easily outweighs the micro performance hit.</p>

<p>As one consequence, the destructor of a dialog is called twice if this dialog's parent is deleted before exec returns when QPointer is not used, as anthonyfieroni pointed out in his edit. This is bound to cause trouble.</p>

<p>fwiw: +1 for this diff</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R95 PIM: Common Support</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7291" rel="noreferrer">https://phabricator.kde.org/D7291</a></div></div><br /><div><strong>To: </strong>winterz, KDE PIM<br /><strong>Cc: </strong>dkurz, anthonyfieroni, KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>