D18604: Implement support for writing rating information for taglib writer
Stefan BrĂ¼ns
noreply at phabricator.kde.org
Thu Jan 31 18:35:35 GMT 2019
bruns added inline comments.
INLINE COMMENTS
> taglibwritertest.cpp:409
> + QTest::addRow("mp3_0")
> + << QStringLiteral("mp3")
> + << QStringLiteral("audio/mpeg3")
What are the differences between the various mp3 test cases?
> taglibwriter.cpp:153
> + //map the rating values of WMP to Baloo rating
> + //0->0, 1->2, 25->4, 50->6, 75->8, 99->10
> + int rating = properties.value(Property::Rating).toInt();
This should probably go into a separate file, togheter with the inverse transform.
This can then be tested independently, i.e.
for rating in {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10} {
r = wmpRating(rating);
Q_ASSERT(rating = balooRating(r)); }
REPOSITORY
R286 KFileMetaData
REVISION DETAIL
https://phabricator.kde.org/D18604
To: astippich, bruns, mgallien
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190131/5497a0a6/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list