<div>ivan created this revision.<br />
ivan added a reviewer: graesslin.<br />
Restricted Application added a project: Plasma.<br />
Restricted Application added a subscriber: plasma-devel.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>From the rest of this method, it is obvious that m_topLevel can<br />
be null in any part of the method - we are checking against it<br />
being null in a few places in the method - both before and after<br />
the affending lines.</p>

<p>Now, there is one place where the check is not applied, and<br />
potentially calls ->window() on the null pointer.</p>

<p>p.s. If there are more places where kwin does clear_or_something(); return;<br />
it could benefit from introducing on_scope_exit and similar tricks - see<br />
Alexandrescu's 'Declarative control flow' presentation.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWIN KWin</div></div></div><br /><div><strong>BRANCH</strong><div><div>ivan/null-check-1</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1346" rel="noreferrer">https://phabricator.kde.org/D1346</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>shadow.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>ivan, graesslin<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>