Review request: MemoryMeta changes and new iPod collection

Matěj Laitl matej at laitl.cz
Mon Jan 16 23:11:08 UTC 2012


On 11. 1. 2012 Bart Cerneels wrote:
> > 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.
> 
> It all looks completely sane and functional to me. Nice touch
> preventing possible threading issues in MemoryMeta.

Okay, thanks for review, merged.

Bart, do you plan to make the UmsCollection changes or should I add it to my 
TODO list?

				Matěj


More information about the Amarok-devel mailing list