Well, before having tried it I have been browsing through the code for a while and I have few comments.<div><br></div><div>- Please explain a little the background, why do you use PCRE instead of QRegExp? it adds a dependency and I'm not sure if it's such a big advantage (I don't know, but there has to be a reason).</div>

<div>- You have been adding the get* prefix to some methods, we generally don't use that to adapt to Qt naming, so don't add those.<br>- Don't use singleShot for delaying slot processing, it's preferred to use QMetaObject::invokeMethod with Qt::QueuedConnection type</div>

<div>- You probably want to use the standard OutputModel you use for storing output. That way you will get some optimizations to make displaying a little faster, I see that you're doing some messy stuff with the model though, so we would see.</div>

<div><br><div class="gmail_quote">On Thu, Aug 26, 2010 at 9:22 PM, Aleix Pol <span dir="ltr"><<a href="mailto:aleixpol@kde.org" target="_blank">aleixpol@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

yup<div><div></div><div><br><br><div class="gmail_quote">On Thu, Aug 26, 2010 at 7:38 PM, Syron <span dir="ltr"><<a href="mailto:mr.syron@googlemail.com" target="_blank">mr.syron@googlemail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div> > well then, at least please merge the current kdevelop so that I can recompile against kdevplatform master.<br>
<br></div>
So you mean...?<br>
$ git checkout grepview2<br>
$ git merge master<br>
$ git push myclone grepview2<div><div></div><div><br>
<br>
-- Syron<br>
<br>
-- <br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org" target="_blank">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>
</div></div></blockquote></div><br></div>