[Libreoffice-commits] core.git: 2 commits - sc/source

Kohei Yoshida kohei.yoshida at gmail.com
Mon Feb 11 20:49:29 PST 2013


 sc/source/filter/xml/editattributemap.cxx |    8 ++++----
 sc/source/filter/xml/xmlcelli.cxx         |    3 ++-
 2 files changed, 6 insertions(+), 5 deletions(-)

New commits:
commit 61867bd74995c0fc35d607427958013998839506
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date:   Mon Feb 11 23:49:47 2013 -0500

    Fix the memory leak. ScEditCell clones the EditTextObject in its ctor.
    
    Change-Id: I8f0f29a504a363ac4359dd4e6ec962976746fecd

diff --git a/sc/source/filter/xml/xmlcelli.cxx b/sc/source/filter/xml/xmlcelli.cxx
index 701e784..3510514 100644
--- a/sc/source/filter/xml/xmlcelli.cxx
+++ b/sc/source/filter/xml/xmlcelli.cxx
@@ -1050,7 +1050,8 @@ void ScXMLTableRowCellContext::PutTextCell( const ScAddress& rCurrentPos,
                         mpEditEngine->QuickInsertField(SvxFieldItem(*it->mpData, EE_FEATURE_FIELD), it->maSelection);
                 }
 
-                pNewCell = new ScEditCell(mpEditEngine->CreateTextObject(), pDoc, pDoc->GetEditPool());
+                boost::scoped_ptr<EditTextObject> pTextObj(mpEditEngine->CreateTextObject());
+                pNewCell = new ScEditCell(pTextObj.get(), pDoc, pDoc->GetEditPool());
             }
         }
         else if ( nCurrentCol > 0 && pOUText && !pOUText->isEmpty() )
commit 664e65f9c291589d9a0cd639fec06bc0575ba6da
Author: Kohei Yoshida <kohei.yoshida at gmail.com>
Date:   Mon Feb 11 23:23:09 2013 -0500

    Forgot to name it consistently...
    
    Change-Id: I331dcd6678c5f781b5535b72d38fd7544ad9b431

diff --git a/sc/source/filter/xml/editattributemap.cxx b/sc/source/filter/xml/editattributemap.cxx
index 174f508..22b4016 100644
--- a/sc/source/filter/xml/editattributemap.cxx
+++ b/sc/source/filter/xml/editattributemap.cxx
@@ -20,7 +20,7 @@ struct {
     sal_uInt16 mnItemID;
     sal_uInt8 mnFlag;
 
-} Entries[] = {
+} aEntries[] = {
 
     { "color", "CharColor", EE_CHAR_COLOR, 0 },
     { "font-charset", "CharFontCharSet", EE_CHAR_FONTINFO, MID_FONT_CHAR_SET },
@@ -76,13 +76,13 @@ ScXMLEditAttributeMap::Entry::Entry(sal_uInt16 nItemID, sal_uInt8 nFlag) :
 
 ScXMLEditAttributeMap::ScXMLEditAttributeMap()
 {
-    size_t n = sizeof(Entries) / sizeof(Entries[0]);
+    size_t n = sizeof(aEntries) / sizeof(aEntries[0]);
     for (size_t i = 0; i < n; ++i)
     {
         maEntries.insert(
             EntriesType::value_type(
-                OUString::createFromAscii(Entries[i].mpAPIName),
-                Entry(Entries[i].mnItemID, Entries[i].mnFlag)));
+                OUString::createFromAscii(aEntries[i].mpAPIName),
+                Entry(aEntries[i].mnItemID, aEntries[i].mnFlag)));
     }
 }
 


More information about the Libreoffice-commits mailing list