<table><tr><td style="">chcuris 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/D14711">View Revision</a></tr></table><br /><div><div><p>Hello David,</p>

<p>I initially checked for mApp->getWindow() separately because when it is null there is no need to go any further, so it seemed simpler to me to directly stop instead of continuing (and couting on people looking at the code guessing that the rest of the code would do nothing in this case);</p>

<p>Furthermore I introduced the variable to query getWindow only once by habit because I prefer to explicit optimisations instead of counting on the compiler (and in this case the optimisation does not happen).</p>

<p>But as you are the project leader (and I thank you for the project), if you want the patch the way you proposed, just confirm me and you'll have it as fast as possible (no need to let that bug lie around).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R875 Falkon</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14711">https://phabricator.kde.org/D14711</a></div></div><br /><div><strong>To: </strong>chcuris, Falkon, drosca<br /><strong>Cc: </strong>drosca, falkon, Falkon, herrold, anmolgautam, SGOrava, iodelay, spoorun, ptabis, navarromorales, cochise, clivej, ach<br /></div>