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

[Jos van den Oever]

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

Ship it!


This looks good.

- Jos van den Oever


On Oct. 18, 2011, 9:07 a.m., Sebastian Trueg wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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/20111026/52eb2e8f/attachment.html>