seamless project model update

Aleix Pol aleixpol at kde.org
Wed Aug 25 05:19:30 UTC 2010


On Sun, Aug 8, 2010 at 4:08 PM, Andreas Pakulat <apaku at gmx.de> wrote:

> On 08.08.10 14:34:35, Aleix Pol wrote:
> > On Sat, Aug 7, 2010 at 4:15 PM, Andreas Pakulat <apaku at gmx.de> wrote:
> > > Actually I mis-remembered what I did myself :) If you look at the code
> in
> > > the model you'll notice that adding items from a background thread is
> > > possible already (invokeMethod is being used) and I think there's even
> a
> > > test for that (though I don't know if it fails without the
> invokeMethod).
> >
> > If you add an element without invokeMethod from a thread that's not the
> same
> > as the view the signals are emitted in the wrong order (well, correct
> order
> > but with the rowCount information in the model already up-to-date which
> > creates the annoying empty cells).
>
> Uhm, I don't think so. We're using BlockingQueuedConnection for a reason
> in the model now. I've actually ran into asserts (as the tests add the
> ModelTest class) because of this.
>
> Andreas
>
> --
> Everything will be just tickety-boo today.
>
> --
> KDevelop-devel mailing list
> KDevelop-devel at kdevelop.org
> https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>

Since now it's possible to add the items from other threads, I decided to
reapply the patch for cmake project model reloading. As you will be able to
see, now it's as smooth as it can be. yay! :) Well, it's just pushed with a
couple of other fixes.

If anybody finds any problem don't hesitate to tell me!

Cheers!
Aleix
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20100825/43a59122/attachment.html>


More information about the KDevelop-devel mailing list