[Kde-bindings] kdebindings/qtruby

Richard Dale Richard_Dale at tipitina.demon.co.uk
Sun Oct 3 12:23:23 UTC 2004


CVS commit by rdale: 


* An optimization in the overloaded method resolution matching causes a crash; 
instead of throwing a ruby exception when a programming error is made. 
If there is only one method found in the Smoke runtime, it assumes that it must  
be the right one and just takes that. 

* For example: 

lay = Qt::HBoxLayout.new(self) 
ed = Qt::LineEdit.new('blah',self) 
# The next line should be: lay.addWidget(ed) 
lay.addLayout(ed) 

* Fixes problem reported by Han Holl 

CCMAIL: kde-bindings at kde.org 


  M +18 -0     ChangeLog   1.115
  M +1 -3      rubylib/qtruby/lib/Qt/qtruby.rb   1.34


--- kdebindings/qtruby/rubylib/qtruby/lib/Qt/qtruby.rb  #1.33:1.34
@@ -422,7 +422,5 @@
                         
                         chosen = nil
-                        if methodIds.length == 1 && method !~ /^operator/
-                                chosen = methodIds[0]
-                        elsif methodIds.length > 0
+                        if methodIds.length > 0
                                 best_match = -1
                                 methodIds.each do

--- kdebindings/qtruby/ChangeLog  #1.114:1.115
@@ -1,4 +1,22 @@
 2004-10-03  Richard Dale  <Richard_Dale at tipitina.demon.co.uk>
 
+ * An optimization in the overloaded method resolution matching causes a crash;
+   instead of throwing a ruby exception when a programming error is made.
+   If there is only one method found in the Smoke runtime, it assumes that it must 
+   be the right one and just takes that.
+
+ * For example:
+
+     lay = Qt::HBoxLayout.new(self)
+     ed = Qt::LineEdit.new('blah',self)
+     # The next line should be: lay.addWidget(ed)
+     lay.addLayout(ed)
+
+  * Fixes problem reported by Han Holl
+
+  CCMAIL: kde-bindings at kde.org
+
+2004-10-03  Richard Dale  <Richard_Dale at tipitina.demon.co.uk>
+
  * A common programming error is to accidently leave off the 'new' method call when
    creating an instance. The QtRuby runtime wasn't correctly trapping an attempt to





More information about the Kde-bindings mailing list