<div class="gmail_quote">On Mon, Jan 17, 2011 at 6:45 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><div></div><div class="h5">On 17.01.11 17:19:21, David Nolden wrote:<br>
> 2011/1/17 Milian Wolff <<a href="mailto:mail@milianw.de">mail@milianw.de</a>>:<br>
> > David Nolden, 17.01.2011:<br>
> >> Hi, it seems like when CodeCompletionWorker etc. were ported to<br>
> >> kdevplatform, someone completely broke the multi-threading. The result<br>
> >> was that for a year now, we are computing all completion items in the<br>
> >> foreground thread (ouch).<br>
> ><br>
> > How come this was not noticed before?<br>
> ><br>
> >> I have fixed this now in kdevplatform and cpp, the other languages<br>
> >> might need a slight adaption. In case you're overriding<br>
> >> CodeCompletionWorker::computeCompletions, then you have to make sure<br>
> >> to match the changed signature of the function you're overriding. The<br>
> >> few other changes should be self-explanatory from the compiler errors.<br>
> >><br>
> >> Now the question is, if we want to backport this to KDevelop 4.2. IMO<br>
> >> the broken multi-threading was a quite serious regression which is<br>
> >> fixed now. The problem is that we're using multi-threading now where<br>
> >> it wasn't used before, which might expose multi-threading bugs we<br>
> >> created within the last year, and which weren't visible without<br>
> >> multi-threading.<br>
> ><br>
> > It seems this requires API changes, hence I'm not in favor of backporting this<br>
> > now that we are in RC state.<br>
> ><br>
> > I know, this sucks, but I don't see any other way.<br>
><br>
> I don't think the binary compatibility is a useful metric for this<br>
> decision, it doesn't make any sense (Unlike kdelibs, we're not a<br>
> library). We should only worry about the stability..<br>
<br>
</div></div>Thats exactly the attitude that makes the situation stay as it is and<br>
why the libraries in kdevplatform are still not used by any project<br>
except kdevelop.<br>
<br>
Apart from that, a change that has unknown side-effects in a stable<br>
release is IMHO just a no-go. Nobody forces us to wait 6 months or more<br>
for the next release if that is your concern. Its also not a regression,<br>
there's been no release with multi-threaded code-completion.<br>
<br>
And apparently for the last year or so, nobody took too much notice of<br>
this to actually try to find out whats wrong.<br>
<br>
Andreas<br>
<font color="#888888"><br>
--<br>
You will experience a strong urge to do good; but it will pass.<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><div>Yes, let's just try to release some kdev4.3 soon :)</div><div><br></div><div>Aleix</div>