<table><tr><td style="">mgallien 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/D16579">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16579#428835" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D16579#428835</a>, <a href="https://phabricator.kde.org/p/smithjd/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@smithjd</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>The change was ack'ed in this review, and this review closed by the commit.</p>

<p>There was no code that required further review, only binary changes. The posted concerns were addressed by <a href="https://phabricator.kde.org/D18826" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D18826</a>.</p>

<p>If you really are that concerned that this review is cleaned up, request yourself that the reviewers that requested changes clear their request. I don't need to ask them to when the request is no longer valid. Otherwise, please at least read the full review closely before complaining about events you weren't consulted on or involved with, and have imo little reason to force a re-visit of.</p></div>
</blockquote>

<p>Hello, the tone of your comment is not appropriate.</p>

<p>Please avoid this.</p>

<p>I agree with <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a>, you should have asked both commenters if they were in agreement with the content and scope of the review before landing it. This is the way reviews work.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16579">https://phabricator.kde.org/D16579</a></div></div><br /><div><strong>To: </strong>smithjd, astippich, bruns, mgallien<br /><strong>Cc: </strong>ngraham, bruns, astippich, kde-frameworks-devel, Baloo, gennad, domson, ashaposhnikov, michaelh, spoorun, abrahams<br /></div>