diff options
author | jailletc36 <christophe.jaillet@wanadoo.fr> | 2012-11-04 16:57:29 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2012-11-06 14:14:34 +0000 |
commit | dda117265966c44f36bcfd8979b7b5aa45318752 (patch) | |
tree | 6242e376fe7ee9532d75d370b16a7b2f5f5ec170 | |
parent | 282d0a2109b6909df778bafa0ef37a8baa972a80 (diff) |
cppCheck: Various clean-up
xfdrawstyle.cxx: correct a copy/paste typo in constructor
number.cxx: remove an unused variable
inftxt.hxx: redundant assignment (pWrongList is assigned twice)
Change-Id: I148e23272ffa169ef16052c475e5f24a87383e5b
Signed-off-by: jailletc36 <christophe.jaillet@wanadoo.fr>
Reviewed-on: https://gerrit.libreoffice.org/983
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
-rw-r--r-- | lotuswordpro/source/filter/xfilter/xfdrawstyle.cxx | 2 | ||||
-rw-r--r-- | sw/source/core/doc/number.cxx | 2 | ||||
-rw-r--r-- | sw/source/core/text/inftxt.hxx | 3 |
3 files changed, 3 insertions, 4 deletions
diff --git a/lotuswordpro/source/filter/xfilter/xfdrawstyle.cxx b/lotuswordpro/source/filter/xfilter/xfdrawstyle.cxx index 5a6bf9013126..bd0fce29f98b 100644 --- a/lotuswordpro/source/filter/xfilter/xfdrawstyle.cxx +++ b/lotuswordpro/source/filter/xfilter/xfdrawstyle.cxx @@ -76,7 +76,7 @@ XFDrawStyle::XFDrawStyle() m_fArrowStartSize = 0.3; m_fArrowEndSize = 0.3; m_bArrowStartCenter = sal_False; - m_bArrowStartCenter = sal_False; + m_bArrowEndCenter = sal_False; } XFDrawStyle::~XFDrawStyle() diff --git a/sw/source/core/doc/number.cxx b/sw/source/core/doc/number.cxx index aa23a2167412..b6a71fe65891 100644 --- a/sw/source/core/doc/number.cxx +++ b/sw/source/core/doc/number.cxx @@ -773,7 +773,6 @@ String SwNumRule::MakeRefNumString( const SwNodeNum& rNodeNum, String aPrevStr = MakeNumString( pWorkingNodeNum->GetNumberVector(), sal_True, sal_False, MAXLEVEL, &aExtremities); - int nLen = aPrevStr.Len(); int nStrip = 0; sal_Unicode c; @@ -789,7 +788,6 @@ String SwNumRule::MakeRefNumString( const SwNodeNum& rNodeNum, { aPrevStr.Erase( 0, nStrip ); aExtremities.nPrefixChars -= nStrip; - nLen -= nStrip; } if ((bFirstIteration || bOldHadPrefix) && diff --git a/sw/source/core/text/inftxt.hxx b/sw/source/core/text/inftxt.hxx index c4cb8d79721b..d750eb3c1c3c 100644 --- a/sw/source/core/text/inftxt.hxx +++ b/sw/source/core/text/inftxt.hxx @@ -404,7 +404,8 @@ protected: SwTxtPaintInfo() { pFrm = 0; pWrongList = 0; pGrammarCheckList = 0; pSmartTags = 0; pSpaceAdd = 0; pBrushItem = ((SvxBrushItem*)-1);} #else - SwTxtPaintInfo() { pFrm = 0; pWrongList = 0; pGrammarCheckList = 0; pWrongList = 0; pSmartTags = 0; pSpaceAdd = 0; pBrushItem = 0;} + SwTxtPaintInfo() { pFrm = 0; pWrongList = 0; pGrammarCheckList = 0; pSmartTags = 0; pSpaceAdd = 0; + pBrushItem = 0;} #endif public: SwTxtPaintInfo( const SwTxtPaintInfo &rInf ); |