[Nepomuk] Review Request: Correct usage of Vorbis tags "artist" and "performer"

[Sebastian Trueg]

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

(Updated Oct. 18, 2011, 9:07 a.m.)


Review request for Nepomuk and Strigi.


Changes
-------

I suppose I was tired when I uploaded the last patch. This patch fixes the issue properly and is better readable because it does

if(!artist.empty())
  if(!performer.empty())

instead of

if(!artist.empty())
  if(performer.empty())


Description
-------

The Vorbis comment specs states:

ARTIST: "The artist generally considered responsible for the work. In popular music this is usually the performing band or singer. For classical music it would be the composer. For an audio book it would be the author of the original text."

PERFORMER: "The artist(s) who performed the work. In classical music this would be the conductor, orchestra, soloists. In an audio book it would be the actor who did the reading. In popular music this is typically the same as the ARTIST and is omitted."

This patch tries to follow that suggestion by using nmm:performer and nmm:composer accordingly.


Diffs (updated)
-----

  lib/endanalyzers/flacendanalyzer.cpp ec885f6 
  lib/throughanalyzers/oggthroughanalyzer.cpp d70f654 

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


Testing
-------

Tried indexing ogg files. The result is correct.


Thanks,

Sebastian Trueg

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/nepomuk/attachments/20111018/5575864d/attachment.html>