<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Aug 11, 2014 at 6:09 PM, Marco Martin <span dir="ltr"><<a href="mailto:notmart@gmail.com" target="_blank">notmart@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Friday 08 August 2014, Eike Hein wrote:<br>
> Hi,<br>
><br>
> many things in Plasma use KDirWatch to keep an eye on changes,<br>
> e.g. the theming and look-and-feel systems, and they usually<br>
> go through the KDirWatch::self() instance.<br>
><br>
> The gotcha with using the shared instance is that slots<br>
> connected to its signals then have to make sure to discard<br>
> notifications about files they don't actually care about but<br>
> *others* do, and Plasma code currently isn't rigorous on that.<br>
> While the theming code is smart about it, I just spotted a<br>
> slot in the look-and-feel code that isn't.<br>
<br>
</div>fixed that one<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Marco Martin<br>
</font></span><div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>Should we maybe agree that KDirWatch::self() is harmful in the context of plasma?</div><div><br></div><div>Aleix </div></div>

</div></div>