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

List:       kde-commits
Subject:    kdenonbeta
From:       Rob Buis <rwlbuis () xs4all ! nl>
Date:       2005-09-02 22:41:43
Message-ID: 1125700903.231036.9622.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 456463 by buis:

Some memleak fixes.


 M  +1 -1      kdom/core/AttrImpl.cpp  
 M  +4 -1      kdom/core/NamedAttrMapImpl.cpp  
 M  +2 -1      ksvg2/svg/SVGLengthListImpl.cpp  


--- trunk/kdenonbeta/kdom/core/AttrImpl.cpp #456462:456463
@@ -158,7 +158,7 @@
         return 0;
 
     if(m_prefix && m_prefix->length())
-        return new DOMStringImpl(m_prefix->string() + QString::fromLatin1(":") + \
ret->string()); +        return new DOMStringImpl(m_prefix->string() + \
QString::fromLatin1(":") + DOMString(ret).string());  
     return ret;
 }
--- trunk/kdenonbeta/kdom/core/NamedAttrMapImpl.cpp #456462:456463
@@ -296,7 +296,7 @@
 
             DOMStringImpl *prevValue = m_attrs[i].value(); 
             m_attrs[i].setValue(value->copy(), m_ownerElement);
-    
+ 
             dispatchAttrMutationEvent(m_ownerElement, prevValue, value, \
m_attrs[i].name(), MODIFICATION);  dispatchSubtreeModifiedEvent();
 
@@ -443,7 +443,10 @@
 {
     DocumentImpl *document = (m_ownerElement ? m_ownerElement->ownerDocument() : 0);
     if(!document || !document->hasListenerType(DOMATTRMODIFIED_EVENT))
+    {
+        if(name) name->deref();
         return;
+    }
 
     DOMStringImpl *eventType = new DOMStringImpl("MutationEvents");
     eventType->ref();
--- trunk/kdenonbeta/ksvg2/svg/SVGLengthListImpl.cpp #456462:456463
@@ -43,7 +43,8 @@
     for(unsigned int i = 0;i < lengths.count();i++)
     {
         SVGLengthImpl *length = new SVGLengthImpl(context, mode);
-        length->setValueAsString(new KDOM::DOMStringImpl(lengths[i]));
+        KDOM::DOMString str(lengths[i]);
+        length->setValueAsString(str.handle());
         appendItem(length);
     }
 }


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

Configure | About | News | Add a list | Sponsored by KoreLogic