<table><tr><td style="">kossebau accepted this revision.<br />kossebau added a comment.<br />This revision is now accepted and ready to land.
</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/D15326">View Revision</a></tr></table><br /><div><div><p>Patch looks good to me also on semantic reading. This should remove the crashes we have seen, and make the code less unusual when it comes to QWidget object handling.</p>

<p>As I think the mentioned bug is related and have seen the same backtrace meanwhile again as well, please add a  line</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">BUG: 398615</pre></div>

<p>to the summary of this review request, so arc (if you use it) will pick up that string when amending the patch with the latest text from here, and the KDE server commit push hooks then do the automatic closing of that bug once they process that commit message (just telling in case you do not know yet) :)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D15326</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15326">https://phabricator.kde.org/D15326</a></div></div><br /><div><strong>To: </strong>antonanikin, KDevelop, kossebau<br /><strong>Cc: </strong>kfunk, kossebau, vkorneev, kdevelop-devel, antismap, iodelay, vbspam, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>