[Kde-bindings] KDE/kdebindings/qtruby

Richard Dale Richard_Dale at tipitina.demon.co.uk
Sun Sep 17 20:37:23 UTC 2006


SVN commit 585710 by rdale:

* Fixed bug reported by Caleb Tennis where temporary QString const arguments
  were only being deleting after marshalling if they weren't const types.

CCMAIL: kde-bindings at kde.org



 M  +5 -0      ChangeLog  
 M  +2 -1      rubylib/qtruby/handlers.cpp  


--- trunk/KDE/kdebindings/qtruby/ChangeLog #585709:585710
@@ -1,3 +1,8 @@
+2006-09-17  Richard Dale  <rdale at foton.es>
+
+	* Fixed bug reported by Caleb Tennis where temporary QString const arguments
+	  were only being deleting after marshalling if they weren't const types.
+
 2006-09-15  Richard Dale  <rdale at foton.es>
 
 	* Fixed bug reported by Henri Sundelin where Qwt classes in .ui files weren't
--- trunk/KDE/kdebindings/qtruby/rubylib/qtruby/handlers.cpp #585709:585710
@@ -839,8 +839,9 @@
 				rb_str_cat2(*(m->var()), StringValuePtr(temp));
 			}
 	
-			if(s && m->type().isConst() && m->cleanup())
+			if (s != 0 && m->cleanup()) {
 				delete s;
+			}
 		}
 		break;
 



More information about the Kde-bindings mailing list