Re: Review Request 124082: Estimate time remaining for FileContentIndexer

2015-06-19 Thread Pinak Ahuja
> On June 18, 2015, 6:39 p.m., Vishesh Handa wrote: > > src/file/timeestimator.h, line 30 > > > > > > Are you sure you're using this class? Right now only in the test but when the new architecture is implemented

Re: Review Request 124082: Estimate time remaining for FileContentIndexer

2015-06-19 Thread Pinak Ahuja
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124082/ --- (Updated June 20, 2015, 6:53 a.m.) Review request for Baloo and Vishesh H

Re: Review Request 124124: Add a basic Monitor with a QML UI

2015-06-19 Thread Pinak Ahuja
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124124/ --- (Updated June 19, 2015, 7:46 a.m.) Status -- This change has been ma

Re: Review Request 124124: Add a basic Monitor with a QML UI

2015-06-19 Thread Pinak Ahuja
> On June 18, 2015, 6:28 p.m., Vishesh Handa wrote: > > Seems interesting. > > > > I'm guessing this is the same code as your github. Yes, plus a few improvements. - Pinak --- This is an automatically generated e-mail. To reply, visit: