More Info

Revision 821449

(Back to digest)
 

Bug Fixes in Multimedia

Mark Kretschmann committed changes in /trunk/extragear/multimedia/amarok/src:

Refactor Meta::Observer so that we automatically unsubscribe all Meta subscriptions in the dtor.

This fixes many issues, e.g. you are now actually able to listen to more than 5 tracks in a row without crashes ;)

Also, it makes the code simpler, since we don't have to do manual bookkeeping.

The important API change is that you now subscribe/unsubscribe to Meta objects with subscribeTo(MetaPtr) / unsubscribeTo(MetaPtr); these two methods are built into the Observer base.

This was quite a bit of work, but I think it was worth it :)

File Changes

Modified 10 files
/trunk/extragear/multimedia/amarok/src
 
/Osd.cpp
 
/covermanager/CoverManager.cpp
 
/meta/Meta.cpp
 
/meta/Meta.h
 
/playlist/PlaylistModel.cpp
 
/widgets/TrackTooltip.cpp
 
/meta/proxy/MetaProxy_p.h
 
/servicebrowser/magnatunestore/MagnatunePurchaseDialog.cpp
 
/context/engines/current/CurrentEngine.cpp
 
/servicebrowser/lastfm/meta/MultiPlayableCapabilityImpl_p.h
10 files changed in total