[Kde-bindings] KDE/kdebindings/ruby/qtruby

Richard Dale Richard_Dale at tipitina.demon.co.uk
Tue Aug 12 09:58:18 UTC 2008


SVN commit 845669 by rdale:

* The new qobject_cast() method was wrongly changing the smoke classId of
  the Ruby source instance. This was messing up method calls such as
  empty?, and fixes a problem reported by Stefano Crocco (thanks again
  Stefano for all these reports!).

CCMAIL: kde-bindings at kde.org



 M  +7 -0      ChangeLog  
 M  +0 -2      src/qtruby.cpp  


--- trunk/KDE/kdebindings/ruby/qtruby/ChangeLog #845668:845669
@@ -1,3 +1,10 @@
+2008-08-12  Richard Dale  <richard.j.dale at gmail.com>
+
+	* The new qobject_cast() method was wrongly changing the smoke classId of
+	  the Ruby source instance. This was messing up method calls such as
+	  empty?, and fixes a problem reported by Stefano Crocco (thanks again
+	  Stefano for all these reports!).
+
 2008-08-11  Richard Dale  <richard.j.dale at gmail.com>
 
 	* Added a qobject_cast() method that uses qt_metacast() to cast one 
--- trunk/KDE/kdebindings/ruby/qtruby/src/qtruby.cpp #845668:845669
@@ -862,8 +862,6 @@
 		return Qnil;
 	}
 
-	o->classId = (int) mi->index;
-
 	QObject* qobj = (QObject*) o->smoke->cast(o->ptr, o->classId, o->smoke->idClass("QObject").index);
 	if (qobj == 0) {
 		return Qnil;



More information about the Kde-bindings mailing list