[Kde-bindings] Qyoto: Movable QDockWidgets

[Arno Rehn]

Am Sonntag, 22. Oktober 2006 12:21 schrieb Richard Dale:
> On Sunday 22 October 2006 09:59, Arno Rehn wrote:
> > Am Samstag, 21. Oktober 2006 23:10 schrieb Richard Dale:
> > > On Wednesday 18 October 2006 22:47, Arno Rehn wrote:
> > > > > I had a look at which virtual methods were being overriden, and it
> > > > > was only ever QObject::metaObject(). I checked that the QDockWidget
> > > > > features were correct and they were set to '7', which should mean
> > > > > the widget is movable ok. So I can't think of what it can be at the
> > > > > moment, or how to find out.
> > > >
> > > > I implemented my ugly hack and it works for now. It's definitely not
> > > > a good solution and if someone has a better one, please check it in!
> > > > But I think at the moment we should focus on other things like D-BUS.
> > > > I haven't tested yet if it works with my last commits, so this will
> > > > be my next task.
> > >
> > > I've found if I disable virtual method overrides in
> > > QyotoSmokeBinding::callMethod() it works fine. And it looked like the
> > > only method being overriden was QObject::metaObject(), and so there
> > > must be something wrong with the QMetaObject returned I suppose.
> >
> > For me it doesn't work either. I commented out my hack and the part in
> > QyotoSmokeBinding where qt_metacall is overridden. The dock widget isn't
> > movable. What parts did you disable in QyotoSmokeBinding::callMethod() ?
>
> I added an unconditional 'return false':
>
> [CODE]

The metaObject is created correctly, it's a problem of the return value of 
QyotoSmokeBinding::callMethod().
Just change the last part:
		VirtualMethodCall c(smoke, method, args, obj, overridenMethod);
		c.next();
		return true;
to
		VirtualMethodCall c(smoke, method, args, obj, overridenMethod);
		c.next();
		return false;
and it will work. I think there is a problem that we return nearly 
always 'true', even if the method isn't overridden.

-- 
Arno Rehn
arno at arnorehn.de