[PATCH] silence a few WaE: private field 'foo' is not used

Matúš Kukan (via_Code_Review) gerrit at gerrit.libreoffice.org
Mon Jan 7 11:13:22 PST 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1566

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/66/1566/1

silence a few WaE: private field 'foo' is not used

Change-Id: I96f5ceda1ae833c5ce58bd846763c003181d55a1
---
M forms/source/component/errorbroadcaster.cxx
M sc/source/filter/oox/stylesbuffer.cxx
M slideshow/source/engine/transitions/fanwipe.cxx
3 files changed, 5 insertions(+), 0 deletions(-)



diff --git a/forms/source/component/errorbroadcaster.cxx b/forms/source/component/errorbroadcaster.cxx
index 8c8baac..f33cd98 100644
--- a/forms/source/component/errorbroadcaster.cxx
+++ b/forms/source/component/errorbroadcaster.cxx
@@ -45,6 +45,7 @@
     //---------------------------------------------------------------------
     OErrorBroadcaster::~OErrorBroadcaster( )
     {
+        (void) m_rBHelper;
         OSL_ENSURE( m_rBHelper.bDisposed || m_rBHelper.bInDispose,
             "OErrorBroadcaster::~OErrorBroadcaster: not disposed!" );
         // herein, we don't have a chance to do the dispose ourself ....
diff --git a/sc/source/filter/oox/stylesbuffer.cxx b/sc/source/filter/oox/stylesbuffer.cxx
index b7cc8db..9aadf33 100644
--- a/sc/source/filter/oox/stylesbuffer.cxx
+++ b/sc/source/filter/oox/stylesbuffer.cxx
@@ -700,6 +700,7 @@
     maUsedFlags( !bDxf ),
     mbDxf( bDxf )
 {
+    (void) mbDxf;
 }
 
 Font::Font( const WorkbookHelper& rHelper, const FontModel& rModel ) :
@@ -708,6 +709,7 @@
     maUsedFlags( true ),
     mbDxf( false )
 {
+    (void) mbDxf;
 }
 
 void Font::importAttribs( sal_Int32 nElement, const AttributeList& rAttribs )
@@ -1675,6 +1677,7 @@
 
 void Border::importDxfBorder( sal_Int32 nElement, SequenceInputStream& rStrm )
 {
+    (void) mbDxf;
     OSL_ENSURE( mbDxf, "Border::importDxfBorder - missing conditional formatting flag" );
     if( BorderLineModel* pBorderLine = getBorderLine( nElement ) )
     {
diff --git a/slideshow/source/engine/transitions/fanwipe.cxx b/slideshow/source/engine/transitions/fanwipe.cxx
index fd3792c..e1f42b7 100644
--- a/slideshow/source/engine/transitions/fanwipe.cxx
+++ b/slideshow/source/engine/transitions/fanwipe.cxx
@@ -51,6 +51,7 @@
     }
     else
     {
+        (void) m_fanIn;
         OSL_ASSERT( ! m_fanIn );
         res.transform(basegfx::tools::createScaleTranslateB2DHomMatrix(0.5, 1.0, 0.5, 1.0));
     }

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I96f5ceda1ae833c5ce58bd846763c003181d55a1
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Matúš Kukan <matus.kukan at gmail.com>



More information about the LibreOffice mailing list