Review request: MemoryMeta changes and new iPod collection

Matěj Laitl laitlmat at fjfi.cvut.cz
Tue Jan 10 13:55:53 UTC 2012


Hi list and mainly Bart,
please review memorymeta-tweaks branch [1] in my personal git clone which 
contains changes that I needed for iPod collection rewrite.

[1] 
http://quickgit.kde.org/?p=clones%2Famarok%2Flaitl%2Famarok.git&a=shortlog&h=refs/heads/memorymeta-
tweaks

UmsCollection would get support for following for free is memorymeta-tweaks is 
merged:
 * album artists (was hard-coded to track artists previously)
 * less memory leaks due to circular referencing removal
 * capability forwarding: editing tags thanks to EditCapability.

ChangeLog is added for above entries.

With a small bit of work, UmsCollection could get support for:
 * album covers if MetaFile::FileAlbum::{image(),hasImage()} is implemented
 * compilations if MetaFile::FileAlbum::isCompilation() is implemented (see 
ArtistHelper::bestGuessAlbumArtist())
 * track removing reflected in collection browser, call 
MapChanger::removeTrack() perhaps somewhere in 
UmsCollectionLocation::removeUrlsFromCollection()
 * track metadata changes reflected in collection browser if you observe tracks 
in UmsCollection and then call MapChanger::trackChanged() in 
metadataChanged(). You should emit updated() when trackChanged() returns true. 
Beware that it cannot currently cope with changes to track uidUrl().

ChangeLog is not updated with above entries as the changes are not user-
visible anywhere in Amarok yet.

All of the MemoryMeta changes are already used & tested in iPod collection 
rewrite that can be found in the ipod-rewrite branch [2], which is nearly 
merge-ready.

[2] 
http://quickgit.kde.org/?p=clones%2Famarok%2Flaitl%2Famarok.git&a=shortlog&h=refs/heads/ipod-
rewrite

Regards,
			Matěj Laitl


More information about the Amarok-devel mailing list