[prev in list] [next in list] [prev in thread] [next in thread]
List: amarok-devel
Subject: Re: Review request: MemoryMeta changes and new iPod collection
From: Matěj Laitl <matej () laitl ! cz>
Date: 2012-01-16 23:11:08
Message-ID: 3826003.YnamKu2ZBi () edgy
[Download RAW message or body]
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
_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic