[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bindings
Subject:    [Kde-bindings] KDE/kdebindings/qtruby
From:       Richard Dale <Richard_Dale () tipitina ! demon ! co ! uk>
Date:       2006-09-17 20:37:23
Message-ID: 1158525443.226436.14622.nullmailer () svn ! kde ! org
[Download RAW message or body]

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@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@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@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;
 
_______________________________________________
Kde-bindings mailing list
Kde-bindings@kde.org
https://mail.kde.org/mailman/listinfo/kde-bindings
[prev in list] [next in list] [prev in thread] [next in thread]