<table><tr><td style="">dvratil requested changes to this revision.<br />dvratil added a comment.<br />This revision now requires changes to proceed.
</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/D10255" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p><a href="https://phabricator.kde.org/p/dkurz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@dkurz</a> please open a new phabricator task for your effort, we can discuss your questions there - and generally, it's a good idea to do that before you start something bigger, so we know what's going on and maybe discuss some details before you start working on it. The main reason is to avoid situations where you write a big chunk of code but it does not get merged because it does not do what we think it should or is missing some important bits. Thanks!</p>

<p><a href="https://phabricator.kde.org/p/winterz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@winterz</a>, I now realize another odditty with your code:</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);">if (this != tItem) {
  return tItem;
} else {
  return this;
}</pre></div>

<p>The <tt style="background: #ebebeb; font-size: 13px;">else</tt> branch implies, that <tt style="background: #ebebeb; font-size: 13px;">this == tItem</tt>, in which case your code could be rewritten as</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);">if (this != tItem) {
  return tItem;
} else {
  return tItem;
}</pre></div>

<p>Which basically means that your change does not do anything, Or am I missing something here?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10255" rel="noreferrer">https://phabricator.kde.org/D10255</a></div></div><br /><div><strong>To: </strong>winterz, dvratil<br /><strong>Cc: </strong>dkurz, dvratil, KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss<br /></div>