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

[Commit Hook]

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


This review has been submitted with commit 1c422171fe65f78d2855992a2e2e93445d17824a by Sebastian Trueg to branch master.

- Commit Hook


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/f3977fbc/attachment.html>