[Libreoffice-commits] core.git: 2 commits - rsc/source sc/source
Julien Nabet
serval2412 at yahoo.fr
Wed Feb 13 09:47:47 PST 2013
rsc/source/res/rscclass.cxx | 2 +-
sc/source/ui/view/tabview2.cxx | 5 +----
2 files changed, 2 insertions(+), 5 deletions(-)
New commits:
commit c3aad7456adb8a24e2350c9b50778adefb2d1444
Author: Julien Nabet <serval2412 at yahoo.fr>
Date: Wed Feb 13 18:44:30 2013 +0100
Fix Variable 'bMark' assigned a value never used
See http://nabble.documentfoundation.org/Cppcheck-bMark-is-assigned-a-value-never-used-sc-module-td4037118.html
Change-Id: Ib2c39c3d38d3a34336e9af0e8389c9ab2f8d5df0
diff --git a/sc/source/ui/view/tabview2.cxx b/sc/source/ui/view/tabview2.cxx
index 59eec9b..888e27c 100644
--- a/sc/source/ui/view/tabview2.cxx
+++ b/sc/source/ui/view/tabview2.cxx
@@ -952,9 +952,8 @@ void ScTabView::PaintBlock( bool bReset )
{
ScMarkData& rMark = aViewData.GetMarkData();
SCTAB nTab = aViewData.GetTabNo();
- bool bMark = rMark.IsMarked();
bool bMulti = rMark.IsMultiMarked();
- if (bMark || bMulti)
+ if (rMark.IsMarked() || bMulti)
{
ScRange aMarkRange;
HideAllCursors();
@@ -966,8 +965,6 @@ void ScTabView::PaintBlock( bool bReset )
rMark.GetMultiMarkArea(aMarkRange);
rMark.MarkToSimple();
rMark.SetMarking(bFlag);
-
- bMark = rMark.IsMarked();
}
else
rMark.GetMarkArea(aMarkRange);
commit e25cdb40dff8615b5881750e197715c17a55df58
Author: Julien Nabet <serval2412 at yahoo.fr>
Date: Wed Feb 13 18:08:36 2013 +0100
Reduce scope
Change-Id: I8e62de343021942fd80ed6846bfa5c0b0ee4a347
diff --git a/rsc/source/res/rscclass.cxx b/rsc/source/res/rscclass.cxx
index 648989a..ec2fb11 100644
--- a/rsc/source/res/rscclass.cxx
+++ b/rsc/source/res/rscclass.cxx
@@ -205,7 +205,6 @@ RSCINST RscClass::Create
)
{
sal_uInt32 i;
- CLASS_DATA * ppData;
RSCINST aInst;
RSCINST aMemInst, aDfltI;
@@ -234,6 +233,7 @@ RSCINST RscClass::Create
if( (VAR_POINTER & pVarTypeList[ i ].nVarType)
&& !(VAR_NODATAINST & pVarTypeList[ i ].nVarType) )
{
+ CLASS_DATA * ppData;
if( VAR_EXTENDABLE & pVarTypeList[ i ].nVarType )
{
RSCINST * pInstance = (RSCINST *)
More information about the Libreoffice-commits
mailing list