<div class="gmail_quote">On Fri, Aug 27, 2010 at 2:56 PM, Andreas Pakulat <span dir="ltr"><<a href="mailto:apaku@gmx.de">apaku@gmx.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 27.08.10 14:29:32, Aleix Pol wrote:<br>
> Well, before having tried it I have been browsing through the code for a<br>
> while and I have few comments.<br>
><br>
> - Please explain a little the background, why do you use PCRE instead of<br>
> QRegExp? it adds a dependency and I'm not sure if it's such a big advantage<br>
> (I don't know, but there has to be a reason).<br>
<br>
</div>In particular as that means having to jump through hoops to make stuff work<br>
correctly with non-ascii strings.<br>
<div class="im"><br>
> - Don't use singleShot for delaying slot processing, it's preferred to use<br>
> QMetaObject::invokeMethod with Qt::QueuedConnection type<br>
<br>
</div>Who said that? singleShot is just fine for doing delayed slot processing if<br>
everything runs in the main thread.<br></blockquote><div>::singleShot uses the timer infrastructure AFAIK, so it does more stuff other than just triggering the slot. It might be a microoptimization but I think it's a good practice to use ::invokeMethod</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
Your analyst has you mixed up with another patient.  Don't believe a<br>
thing he tells you.<br>
</font><div><div></div><div class="h5"><br>
--<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</div></div></blockquote></div><br>