> On Sept. 24, 2012, 8:46 a.m., Bart Cerneels wrote: > > ChangeLog, line 17 > > <http://git.reviewboard.kde.org/r/106550/diff/1/?file=86926#file86926line17> > > > > Is that Czech english? ;) > > "instead of keeping then in the database."
Yes, sorry, too late in the night. - Matěj ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106550/#review19376 ----------------------------------------------------------- On Sept. 23, 2012, 11:39 p.m., Matěj Laitl wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106550/ > ----------------------------------------------------------- > > (Updated Sept. 23, 2012, 11:39 p.m.) > > > Review request for Amarok. > > > Description > ------- > > MetaFile::Track: only provide statistics stored in tags > > As agreed in Randa that non-collection tracks shouldn't use external > tables to store statistics. When user doesn't enable writing stats to > files, we don't even read them in order not to annoy the user. > > Also clean up the setter methods in Meta::Track, code sharing+++ > > > Diffs > ----- > > ChangeLog 7b58b74eae5f496dc714a78398133e7c08dd96fb > src/core-impl/meta/file/File.h 650c79d0f3c6af581c117f33382095db5018c63f > src/core-impl/meta/file/File.cpp a3ca501449590d2f43a30a9b288cf51089515bff > src/core-impl/meta/file/File_p.h d29d0948047407d5798dd98a21fd75a8f66223f7 > tests/core-impl/meta/file/TestMetaFileTrack.cpp > 6ee7f3c7e79dd1c18caa3549669c670dc3502686 > tests/core/meta/TestMetaTrack.h 5eb49f0f73266542efb19065cda5acf57be44acc > tests/core/meta/TestMetaTrack.cpp 68bc84f8e6fc3efa6d6c5fd74e41808d01edd5af > > Diff: http://git.reviewboard.kde.org/r/106550/diff/ > > > Testing > ------- > > Works. This is an implementation of what has been agreed on on the > requirements/architecture meetings. Running this through review to make sure > we all want this. > > > Thanks, > > Matěj Laitl > >
_______________________________________________ Amarok-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/amarok-devel
