<table><tr><td style="">franckarrecot marked 2 inline comments as done.<br />franckarrecot added inline comments.
</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/D9015" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D9015#inline-40563" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ervin</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kfileplacesview.cpp:811-813</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This whole change on the if structure there is unrelated to the rest of the patch so please remove it. Beside this seems to change the semantic, we never tested for result being nullptr or not before, we tested for the various actions to be there, you changed that logic completely that smells dangerous to me.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Indeed the structural change could be in an other patch. Not sure to see where it's dangerous tho, previous code testing the nullptr where doing it on a variable eg: empty trash, and only if result variable was equal to empty trash, which had to be differnt from nullptr it would go on, I think this endup being the same with the new change.</p>

<p style="padding: 0; margin: 8px;">If result is different from nullptr AND equal to empty trash is equivalent to result being equal to emptyTrash and emptyTrash being different to nulltpr. The code is not doing anything for result being equal to nullptr alone, so no new behavior AFAIK.</p>

<p style="padding: 0; margin: 8px;">Removing it, maybe in an other commit :-)</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9015" rel="noreferrer">https://phabricator.kde.org/D9015</a></div></div><br /><div><strong>To: </strong>franckarrecot, ervin, renatoo, mlaurent, ngraham<br /><strong>Cc: </strong>Frameworks<br /></div>