<table><tr><td style="">astippich 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/D18819">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/D18819#407360" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D18819#407360</a>, <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> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>The default match mode <a href="https://doc.qt.io/qt-5/qmimedatabase.html#MatchMode-enum" class="remarkup-link" target="_blank" rel="noreferrer">tries both file extension and content</a>; this would make it just do content. So I guess if they don't match, the default match mode uses the file extension rather than content? That seems kinda like the opposite of the behavior you'd want; might be worth a Qt bug report and aldo a comment in the code here explaining why we're doing this.</p>

<p>The change itself seems sane since we're actually doing less, more correct work now.</p>

<p>What more is needed to fix 403902?</p></div>
</blockquote>

<p>The file extension always has priority over the content <a href="https://doc.qt.io/qt-5/qmimedatabase.html#mimeTypeForFile" class="remarkup-link" target="_blank" rel="noreferrer">according to docs</a>.<br />
This is actually more a workaround for the bug (but still the right thing to do nevertheless), we now make sure that Baloo never calls the crashing extractor for the problematic files. The crash inside the extractor actually remains.<br />
I have the real fix for crash actually ready, but it is more a complete rewrite of the taglib extractor.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18819">https://phabricator.kde.org/D18819</a></div></div><br /><div><strong>To: </strong>astippich, ngraham, bruns<br /><strong>Cc: </strong>kde-frameworks-devel, Baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>