D10918: taglibextractor: Refactor for better readability

Alexander Stippich noreply at phabricator.kde.org
Wed Mar 14 20:57:20 UTC 2018


astippich requested changes to this revision.
astippich added a comment.
This revision now requires changes to proceed.


  Looks good imho, just one small comment inline. of course, @mgallien as the maintainer must also give his ok.

INLINE COMMENTS

> taglibextractor.cpp:331
> +    } else {
> +        extractOgg(stream, mimeType, data);
>      }

I think we should check explicitly for the appropriate mimetypes here

REPOSITORY
  R286 KFileMetaData

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

To: michaelh, mgallien, #baloo, #frameworks, astippich
Cc: astippich, ashaposhnikov, michaelh, spoorun, nicolasfella, ngraham, alexeymin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180314/07d98624/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list