----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102336/#review5738 -----------------------------------------------------------
Code looks good to me. But does the riff analyser only handle audio or also video files? - Sebastian On Aug. 16, 2011, 9:19 a.m., Vishesh Handa wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102336/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2011, 9:19 a.m.) > > > Review request for Nepomuk, Strigi, Jos van den Oever, and Sebastian Trueg. > > > Summary > ------- > > Add the required type. I'm not sure if I should be adding nfo:Audio or > nfo:MusicPiece. > > > Diffs > ----- > > plugins/eventplugins/riffeventanalyzer.cpp 69aa927 > > Diff: http://git.reviewboard.kde.org/r/102336/diff > > > Testing > ------- > > Works fine, and the files are getting indexed by Nepomuk as they have the > correct type. > > > Thanks, > > Vishesh > >
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
