[Kde-bindings] [PATCH] Deferred objects, QGraphicsEffect, long long handler

[Melton, Ryan]

Hi Arno,

It appears you implemented one line of Paolo's patch differently than he did for Qt.cpp:

Yours:	unmapPointer(o, o->classId, 0);
Paolos:	unmapPointer(o, o->classId, 0, ptr);

Was this intentional? Or maybe is the following possibly correct?
unmapPointer(o, o->classId, ptr);

Thanks,
Ryan

-----Original Message-----
From: Arno Rehn [mailto:arno at arnorehn.de] 
Sent: Wednesday, July 28, 2010 2:52 PM
To: KDE bindings for other programming languages
Subject: Re: [Kde-bindings] [PATCH] Deferred objects, QGraphicsEffect,long long handler

On Wednesday 21 July 2010 17:42:02 Richard Dale wrote:
> On Sunday, July 11, 2010 06:24:22 pm Paolo Capriotti wrote:
> > Hi all,
> > here are 3 patches that fix some problems I'm having with the current
> > qtruby.
> > 
> > Deferred objects (http://gist.github.com/471683): I already submitted
> > a patch about this problem. This is an updated version for the current
> > svn trunk. The idea (as explained here:
> > http://article.gmane.org/gmane.comp.kde.devel.bindings/5715) is that
> > there is no guarantee that the VALUE associated to an object still
> > contains meaningful information when the object destructor is called.
> > So we cannot use value_obj_info from within Binding::deleted. This
> > patch saves a pointer to the smokeruby_object instance alongside the
> > ruby VALUE in the pointer map.
> > 
> > QGraphicsEffect (http://gist.github.com/471684): A special case
> > similar to the others in that function: a QGraphicsItem owns its
> > QGraphicsEffect's, so they must be marked in the mark phase.
> > 
> > long long handler (http://gist.github.com/471514): There is no handler
> > for the "long long" type. Not sure why this hasn't popped up before,
> > but it causes a crash on my machine when calling
> > QIODevice::bytesAvailable.
> > 
> > Let me know if there are any objections. If not, I'll commit them soon.
> 
> I'm sorry I've been so long in replying, but please commit them if you
> haven't already. Just CC this mailing list so we know it went in we you do
> commit stuff.
I committed the patches myself now so that they make it into KDE 4.5.0. Thanks 
for providing them Paolo! :)

-- 
Arno Rehn
arno at arnorehn.de
_______________________________________________
Kde-bindings mailing list
Kde-bindings at kde.org
https://mail.kde.org/mailman/listinfo/kde-bindings



This message and any enclosures are intended only for the addressee.  Please  
notify the sender by email if you are not the intended recipient.  If you are  
not the intended recipient, you may not use, copy, disclose, or distribute this  
message or its contents or enclosures to any other person and any such actions  
may be unlawful.  Ball reserves the right to monitor and review all messages  
and enclosures sent to or from this email address.