Re: Review Request 125004: Make "querying for remaining time" thread safe

2015-09-01 Thread Stefan Brüns
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125004/#review84723 --- src/file/timeestimator.cpp (line 45)

Re: Bug or Non-sense I don't know where to submit

2015-09-01 Thread Thomas Lübking
PS: off-topic. User questions/comments/feedback ideally goes forum.kde.org Bug reports please go bugs.kde.org Cheers, Thomas Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

Re: Bug or Non-sense I don't know where to submit

2015-09-01 Thread Thomas Lübking
On Dienstag, 1. September 2015 16:40:16 CEST, Olivier Churlaud wrote: Hi, A new feature really disturbs me but I don't know where to submit this feedback, so I do it there. If it's not appropriate, please excuse-me and redirect me to the right place. Since the new Applications update, a glob

Bug or Non-sense I don't know where to submit

2015-09-01 Thread Olivier Churlaud
Hi, A new feature really disturbs me but I don't know where to submit this feedback, so I do it there. If it's not appropriate, please excuse-me and redirect me to the right place. Since the new Applications update, a global setting has been introduced to act on click or double-click. I find

Re: Review Request 125004: Make "querying for remaining time" thread safe

2015-09-01 Thread Pinak Ahuja
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125004/ --- (Updated Sept. 1, 2015, 11:42 a.m.) Review request for Baloo and Vishesh

Re: Review Request 125004: Make "querying for remaining time" thread safe

2015-09-01 Thread Pinak Ahuja
> On Aug. 31, 2015, 8:55 p.m., Vishesh Handa wrote: > > src/file/fileindexscheduler.h, line 106 > > > > > > Do you think it makes more sense for the TimeEstimator class to store > > the buffer? Yes that, makes

Re: Review Request 124919: Re-organize D-Bus interfaces

2015-09-01 Thread Harald Sitter
> On Aug. 31, 2015, 8:45 a.m., Hrvoje Senjan wrote: > > src/dbus/CMakeLists.txt, line 26 > > > > > > This looks like a SiC change to me, no? e.g. plasma-desktop won't build > > with this commit > > Harald Sitter