----------------------------------------------------------- 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
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
