Re: Use of KDirWatch

2014-08-12 Thread Sebastian Kügler
On Tuesday, August 12, 2014 00:10:59 Aleix Pol wrote: > Should we maybe agree that KDirWatch::self() is harmful in the context of > plasma? It's not more harmful than anywhere else, IMO. We should keep an eye out for it in review requests, though. The shared KDirWatch can be useful, and we shoul

Re: Use of KDirWatch

2014-08-11 Thread Aleix Pol
On Mon, Aug 11, 2014 at 6:09 PM, Marco Martin wrote: > On Friday 08 August 2014, Eike Hein wrote: > > Hi, > > > > many things in Plasma use KDirWatch to keep an eye on changes, > > e.g. the theming and look-and-feel systems, and they usually > > go through the KDirWatch::self() instance. > > > >

Re: Use of KDirWatch

2014-08-11 Thread Marco Martin
On Friday 08 August 2014, Eike Hein wrote: > Hi, > > many things in Plasma use KDirWatch to keep an eye on changes, > e.g. the theming and look-and-feel systems, and they usually > go through the KDirWatch::self() instance. > > The gotcha with using the shared instance is that slots > connected t

Use of KDirWatch

2014-08-08 Thread Eike Hein
Hi, many things in Plasma use KDirWatch to keep an eye on changes, e.g. the theming and look-and-feel systems, and they usually go through the KDirWatch::self() instance. The gotcha with using the shared instance is that slots connected to its signals then have to make sure to discard notificat