----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114983/#review47256 -----------------------------------------------------------
Ship it! This is awesome. Ship it! If you want you can also get rid of the IndexCleaner completely. I think ti might be better to move it into the KCM, and only run it when the configuration changes. There isn't much point running it each time. src/file/indexscheduler.cpp <https://git.reviewboard.kde.org/r/114983/#comment33662> I'm a little bit concerned about this because one might decide that the BasicIQ should not check every file on startup. Look at the fileindexer.cpp, there is a check over there to see if initial check is enabled / disabled. - Vishesh Handa On Jan. 12, 2014, 3:43 p.m., Simeon Bird wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/114983/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2014, 3:43 p.m.) > > > Review request for Baloo and Vishesh Handa. > > > Repository: baloo > > > Description > ------- > > Misc. File queue changes - the top edit should perhaps be applied to > IndexScheduler::slotCommitted > as well. > > commit ea35771246890c46f63d30f765a0d19844bd3d63 > Author: Simeon Bird <bla...@gmail.com> > Date: Sat Jan 11 14:57:39 2014 -0500 > > Instead of unconditionally resuming the file queue when the basic queue > is done, call slotScheduleIndexing. > > Otherwise the file queue would start once the basic queue was done, even > on battery. > > commit b6aeea48286e53206726af48dc65da026c3dd7a8 > Author: Simeon Bird <bla...@gmail.com> > Date: Sat Jan 11 14:34:17 2014 -0500 > > Refactor a function in indexscheduler.cpp to reduce duplication > > commit 4aa06719a98f740fb55c51232ac306409a860077 > Author: Simeon Bird <bla...@gmail.com> > Date: Sat Jan 11 14:21:18 2014 -0500 > > Do not call fillQueue when the file queue is created, but when the basic > queue is done. > > We only know what work needs to be done in the file queue once the basic > queue is complete, so call fillQueue at that time, not when the queue is > created. > > Before we could be filling the file queue pointlessly with > tasks to be cleaned, or, if the file queue was empty when > the basic queue started, it might not start when it should have done. > > commit 406a6a4ae4f5facbb33b7de433347d84f77182c5 > Author: Simeon Bird <bla...@gmail.com> > Date: Sat Jan 11 12:44:52 2014 -0500 > > FileIndexingQueue doesn't need to listen to ConfigChanged to refill the > indexing queue, because that is handled in the IndexScheduler already > > > Diffs > ----- > > src/file/fileindexingqueue.h c112c97 > src/file/fileindexingqueue.cpp 0f22caf > src/file/indexscheduler.h 015d23a > src/file/indexscheduler.cpp 79cd5bf > > Diff: https://git.reviewboard.kde.org/r/114983/diff/ > > > Testing > ------- > > Compiled, ran with it. > > > Thanks, > > Simeon Bird > >
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
