[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:       Bart Cerneels <bart.cerneels () kde ! org>
Date:       2012-01-11 10:45:59
Message-ID: CAMnMsSea+VzWMpeduEo-AzSGiM-VwZ7bY05jVpmMS4CK4rPK_A () mail ! gmail ! com
[Download RAW message or body]

On Tue, Jan 10, 2012 at 14:55, Matěj Laitl <laitlmat@fjfi.cvut.cz> wrote:
> 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.

It all looks completely sane and functional to me. Nice touch
preventing possible threading issues in MemoryMeta.


>
> 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
>

I'll test this later.
_______________________________________________
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]