> On Aug. 14, 2012, 8:30 a.m., Bart Cerneels wrote: > > src/core-impl/collections/spotifycollection/SpotifySettings.cpp, line 217 > > <http://git.reviewboard.kde.org/r/105285/diff/3/?file=77322#file77322line217> > > > > You probably should do readAll() here. The data will be fully available > > after finished(). > > > > Unless there is some platform specific reason to do it like this?
No specific reason but I was thinking the resolver binary might be very big, I once used a 100MB file to test downloading. Now it is replaced by readAll(). - Zhengliang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105285/#review17359 ----------------------------------------------------------- On Aug. 13, 2012, 11:24 a.m., Zhengliang Feng wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105285/ > ----------------------------------------------------------- > > (Updated Aug. 13, 2012, 11:24 a.m.) > > > Review request for Amarok. > > > Description > ------- > > Things I've done this week: > * Added a new playlist provider SpotifyPlaylistProvider > * Added playlist class SpotifyPlaylist > * Clear all extra whitespaces > * Figured out how Collection, QueryMaker and Playlist worked > > Things need to be done next week: > * Clean ScriptResolver's interfaces, move all query related interfaces into > Query class > * Replace Controller class with ScriptResolver > * Test SpotifyCollection > * Finish playlist and playlist provider > > > Diffs > ----- > > src/core-impl/collections/CMakeLists.txt c78b920 > src/core-impl/collections/spotifycollection/CMakeLists.txt PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyCollection.h > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyCollection.cpp > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyConfig.h PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyConfig.cpp PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyConfigWidget.ui > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyMeta.h PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyMeta.cpp PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyPlaylist.h PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyPlaylist.cpp > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyPlaylistProvider.h > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyPlaylistProvider.cpp > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyQueryMaker.h > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifyQueryMaker.cpp > PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifySettings.h PRE-CREATION > src/core-impl/collections/spotifycollection/SpotifySettings.cpp > PRE-CREATION > > src/core-impl/collections/spotifycollection/amarok_collection-spotifycollection.desktop > PRE-CREATION > src/core-impl/collections/spotifycollection/support/Controller.h > PRE-CREATION > src/core-impl/collections/spotifycollection/support/Controller.cpp > PRE-CREATION > src/core-impl/collections/spotifycollection/support/QMFunctionTypes.h > PRE-CREATION > src/core-impl/collections/spotifycollection/support/Query.h PRE-CREATION > src/core-impl/collections/spotifycollection/support/Query.cpp PRE-CREATION > src/core-impl/collections/spotifycollection/support/TrackProxy.h > PRE-CREATION > src/core-impl/collections/spotifycollection/support/TrackProxy.cpp > PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/105285/diff/ > > > Testing > ------- > > No test done this week. > > > Thanks, > > Zhengliang Feng > >
_______________________________________________ Amarok-devel mailing list Amarok-devel@kde.org https://mail.kde.org/mailman/listinfo/amarok-devel
