[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