bruns accepted this revision.
bruns added a comment.
This revision is now accepted and ready to land.


  Add the comment, otherwise its fine.

INLINE COMMENTS

> taglibextractor.cpp:843
> +    } else if (mimeType == QLatin1String("audio/x-ms-wma")) {
> +        TagLib::ASF::Tag* asfTags = dynamic_cast<TagLib::ASF::Tag*>(tags);
> +        if (asfTags) {

Can you add a comment here for why the dynamic_cast is used here?

REPOSITORY
  R286 KFileMetaData

BRANCH
  extract_asf

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

To: astippich, mgallien, bruns
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, 
ngraham, bruns, abrahams

Reply via email to