Review Request 113777: Lazy load scriptable service tracks

Anmol Ahuja darthcodus at gmail.com
Mon Nov 25 19:22:57 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113777/
-----------------------------------------------------------

(Updated Nov. 26, 2013, 12:52 a.m.)


Review request for Amarok.


Changes
-------

Set m_loaded in constructor.
Restore playlist download error popups.


Repository: amarok


Description
-------

Prevent all the scriptable services from loading playlists at Amarok launch.


Diffs (updated)
-----

  src/services/scriptable/ScriptableServiceMeta.cpp 8e96785 
  src/services/scriptable/ScriptableServiceMeta.h 2db3951 
  src/core-impl/playlists/types/file/PlaylistFileLoaderJob.cpp 75f4d01 

Diff: http://git.reviewboard.kde.org/r/113777/diff/


Testing
-------


Thanks,

Anmol Ahuja

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20131125/fab83256/attachment-0001.html>


More information about the Amarok-devel mailing list