D18819: Use content to determine mime type

[Alexander Stippich]

astippich added a comment.


  In D18819#407360 <https://phabricator.kde.org/D18819#407360>, @ngraham wrote:
  
  > The default match mode tries both file extension and content <https://doc.qt.io/qt-5/qmimedatabase.html#MatchMode-enum>; 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.
  >
  > The change itself seems sane since we're actually doing less, more correct work now.
  >
  > What more is needed to fix 403902?
  
  
  The file extension always has priority over the content according to docs <https://doc.qt.io/qt-5/qmimedatabase.html#mimeTypeForFile>.
  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.
  I have the real fix for crash actually ready, but it is more a complete rewrite of the taglib extractor.

REPOSITORY
  R293 Baloo

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D18819

To: astippich, ngraham, bruns
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190207/c510f6c3/attachment.html>