Re: Review Request 114983: Misc changes to the file queue

2014-01-12 Thread Simeon Bird
> On Jan. 12, 2014, 4:11 p.m., Vishesh Handa wrote: > > 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 ea

Re: Review Request 114981: Rework slotScheduleIndexing a bit to fix some corner cases

2014-01-12 Thread Simeon Bird
> On Jan. 12, 2014, 4:17 p.m., Vishesh Handa wrote: > > Seems pretty cool. You rock! Cheers! > On Jan. 12, 2014, 4:17 p.m., Vishesh Handa wrote: > > src/file/indexingqueue.cpp, line 55 > > > > > > I seem to hav

Re: Review Request 114981: Rework slotScheduleIndexing a bit to fix some corner cases

2014-01-12 Thread Vishesh Handa
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114981/#review47257 --- Ship it! Seems pretty cool. You rock! src/file/indexingqueu

Re: Review Request 114983: Misc changes to the file queue

2014-01-12 Thread Vishesh Handa
--- 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 g

Review Request 114983: Misc changes to the file queue

2014-01-12 Thread Simeon Bird
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114983/ --- Review request for Baloo and Vishesh Handa. Repository: baloo Descripti

Review Request 114981: Rework slotScheduleIndexing a bit to fix some corner cases

2014-01-12 Thread Simeon Bird
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114981/ --- Review request for Baloo and Vishesh Handa. Repository: baloo Descripti