[PATCH] fdo#39440 - cppcheck cleanliness

Ioan Radu (via Code Review) gerrit at gerrit.libreoffice.org
Sun Dec 30 15:28:19 PST 2012


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1518

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/18/1518/1

fdo#39440 - cppcheck cleanliness

removed some cppcheck warning

Change-Id: I81963001e4737c583764b07bea9abf59a61dead7
---
M lotuswordpro/source/filter/lwpbackgroundstuff.cxx
1 file changed, 2 insertions(+), 5 deletions(-)



diff --git a/lotuswordpro/source/filter/lwpbackgroundstuff.cxx b/lotuswordpro/source/filter/lwpbackgroundstuff.cxx
index d26e0b3..a38d419 100644
--- a/lotuswordpro/source/filter/lwpbackgroundstuff.cxx
+++ b/lotuswordpro/source/filter/lwpbackgroundstuff.cxx
@@ -169,11 +169,8 @@
     XFBGImage* pXFBGImage = new XFBGImage();
     pXFBGImage->SetImageData(pImageBuff, nSize);
 
-    if (pImageBuff)
-    {
-        delete [] pImageBuff;
-        pImageBuff = NULL;
-    }
+    delete [] pImageBuff;
+    pImageBuff = NULL;
 
 
     pXFBGImage->SetRepeate();

-- 
To view, visit https://gerrit.libreoffice.org/1518
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I81963001e4737c583764b07bea9abf59a61dead7
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Ioan Radu <ioan.radu.g at gmail.com>



More information about the LibreOffice mailing list