Review Request 112266: Fix reading Album Artist / Compilation / Disc Number in APE tags

[Bruno Léon]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112266/
-----------------------------------------------------------

(Updated Aug. 30, 2013, 1:33 p.m.)


Review request for Amarok.


Changes
-------

What is the decision regarding this issue ?


Description
-------

Fix reading of Album Artist and Compilation tag in APE tags.
Add support for reading Disc Number in APE tags.


Diffs
-----

  shared/tag_helpers/APETagHelper.cpp c628694 

Diff: http://git.reviewboard.kde.org/r/112266/diff/


Testing
-------

Tested with Musepack files (that do use APE tags)


File Attachments (updated)
----------------

Taglib output
  http://git.reviewboard.kde.org/media/uploaded/files/2013/08/25/taglib_output


Thanks,

Bruno Léon

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20130830/0c19bd43/attachment.html>