[Libreoffice] [PATCH] cppcheck cleaning in writer

[Julien Nabet]

Hello,

Here is a patch for 2 memleaks + 1  assigned a value that is never used
Compiling was ok

Julien
(LGPLv3+ / MPL)

commit 29e339267845268669dbecb036eb11fbe539d9ef
Author: Julien Nabet <serval2...@yahoo.fr>
Date:   Sun Dec 26 17:04:32 2010 +0100

    cppcheck cleaning in writer

diff --git a/sw/source/core/crsr/swcrsr.cxx b/sw/source/core/crsr/swcrsr.cxx
index 3dec37b..3842154 100644
--- a/sw/source/core/crsr/swcrsr.cxx
+++ b/sw/source/core/crsr/swcrsr.cxx
@@ -997,6 +997,7 @@ ULONG SwCursor::FindAll( SwFindParas& rParas,
             }
             else
                 DeleteMark();
+            delete pSav;
             return 0;
         }
 
@@ -1051,6 +1052,7 @@ ULONG SwCursor::FindAll( SwFindParas& rParas,
             }
             else
                 DeleteMark();
+            delete pSav;
             return 0;
         }
         // es  wurde ein- bis mehrmals gefunden. Das steht alles
diff --git a/sw/source/core/crsr/trvltbl.cxx b/sw/source/core/crsr/trvltbl.cxx
index f466c34..fc6266e 100644
--- a/sw/source/core/crsr/trvltbl.cxx
+++ b/sw/source/core/crsr/trvltbl.cxx
@@ -120,8 +120,7 @@ BOOL SwCrsrShell::GoNextCell( BOOL bAppendLine )
 BOOL SwCrsrShell::GoPrevCell()
 {
     BOOL bRet = FALSE;
-    const SwTableNode* pTblNd;
-    if( IsTableMode() || 0 != ( pTblNd = IsCrsrInTbl() ))
+    if( IsTableMode() || IsCrsrInTbl() )
     {
         SwCursor* pCrsr = pTblCrsr ? pTblCrsr : pCurCrsr;
         SwCallLink aLk( *this );               // Crsr-Moves ueberwachen,

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice