loplugin:casttovoid: svtools

Change-Id: Iae781dd1f303b905653ccd1e113cfdbb7145819e

5 files changed, 21 insertions, 47 deletions

diff --git a/svtools/source/misc/acceleratorexecute.cxx b/svtools/source/misc/acceleratorexecute.cxx
index 207e9d4448b8..3d2bca6a156c 100644
--- a/svtools/source/misc/acceleratorexecute.cxx
+++ b/svtools/source/misc/acceleratorexecute.cxx
@@ -374,8 +374,8 @@ css::uno::Reference< css::ui::XAcceleratorConfiguration > AcceleratorExecute::st
     {
         sModule = xModuleDetection->identify(xFrame);
     }
-    catch(const css::uno::RuntimeException&rEx)
-        { (void) rEx; throw; }
+    catch(const css::uno::RuntimeException&)
+        { throw; }
     catch(const css::uno::Exception&)
         { return css::uno::Reference< css::ui::XAcceleratorConfiguration >(); }
 
diff --git a/svtools/source/table/gridtablerenderer.cxx b/svtools/source/table/gridtablerenderer.cxx
index 7df61b8db309..d9af224e8153 100644
--- a/svtools/source/table/gridtablerenderer.cxx
+++ b/svtools/source/table/gridtablerenderer.cxx
@@ -233,13 +233,15 @@ namespace svt { namespace table
         rRenderContext.DrawLine(_rArea.BottomRight(), _rArea.TopRight());
 
         rRenderContext.Pop();
-        (void)_bIsColHeaderArea;
-        (void)_bIsRowHeaderArea;
     }
 
 
-    void GridTableRenderer::PaintColumnHeader(ColPos _nCol, bool _bActive, bool _bSelected, vcl::RenderContext& rRenderContext,
-                                              const tools::Rectangle& _rArea, const StyleSettings& _rStyle)
+    void GridTableRenderer::PaintColumnHeader(
+        ColPos _nCol,
+        bool, // _bActive: no special painting for the active column at the moment
+        bool, // _bSelected: selection for column header not yet implemented
+        vcl::RenderContext& rRenderContext,
+        const tools::Rectangle& _rArea, const StyleSettings& _rStyle)
     {
         rRenderContext.Push(PushFlags::LINECOLOR);
 
@@ -293,12 +295,6 @@ namespace svt { namespace table
         }
 
         rRenderContext.Pop();
-
-        (void)_bActive;
-        // no special painting for the active column at the moment
-
-        (void)_bSelected;
-        // selection for column header not yet implemented
     }
 
 
@@ -550,9 +546,8 @@ namespace svt { namespace table
     }
 
 
-    void GridTableRenderer::HideCellCursor( vcl::Window& _rView, const tools::Rectangle& _rCursorRect)
+    void GridTableRenderer::HideCellCursor( vcl::Window& _rView, const tools::Rectangle&)
     {
-        (void)_rCursorRect;
         _rView.HideFocus();
     }
 
diff --git a/svtools/source/table/mousefunction.cxx b/svtools/source/table/mousefunction.cxx
index 0b947b9f5bfb..631841fd0085 100644
--- a/svtools/source/table/mousefunction.cxx
+++ b/svtools/source/table/mousefunction.cxx
@@ -77,7 +77,6 @@ namespace svt { namespace table
             ShowTrackFlags::Split | ShowTrackFlags::TrackWindow
         );
 
-        (void)i_event;
         return ContinueFunction;
     }
 
diff --git a/svtools/source/table/tablecontrol.cxx b/svtools/source/table/tablecontrol.cxx
index 7fca9670e48f..cf97e3f9f7b2 100644
--- a/svtools/source/table/tablecontrol.cxx
+++ b/svtools/source/table/tablecontrol.cxx
@@ -287,9 +287,8 @@ namespace svt { namespace table
     }
 
 
-    Reference<XAccessible> TableControl::CreateAccessibleControl( sal_Int32 _nIndex )
+    Reference<XAccessible> TableControl::CreateAccessibleControl( sal_Int32 )
     {
-        (void)_nIndex;
         SAL_WARN( "svtools", "TableControl::CreateAccessibleControl: to be overwritten!" );
         return nullptr;
     }
@@ -379,9 +378,8 @@ namespace svt { namespace table
     }
 
 
-    OUString TableControl::GetRowDescription( sal_Int32 _nRow) const
+    OUString TableControl::GetRowDescription( sal_Int32) const
     {
-        (void)_nRow;
         return OUString( "row description" );
     }
 
@@ -394,9 +392,8 @@ namespace svt { namespace table
     }
 
 
-    OUString TableControl::GetColumnDescription( sal_uInt16 _nColumn) const
+    OUString TableControl::GetColumnDescription( sal_uInt16) const
     {
-        (void)_nColumn;
         return OUString( "col description" );
     }
 
@@ -581,7 +578,7 @@ namespace svt { namespace table
     }
 
 
-    void TableControl::FillAccessibleStateSetForCell( ::utl::AccessibleStateSetHelper& _rStateSet, sal_Int32 _nRow, sal_uInt16 _nColumnPos ) const
+    void TableControl::FillAccessibleStateSetForCell( ::utl::AccessibleStateSetHelper& _rStateSet, sal_Int32 _nRow, sal_uInt16 ) const
     {
         if ( IsRowSelected( _nRow ) )
             _rStateSet.AddState( AccessibleStateType::SELECTED );
@@ -595,23 +592,17 @@ namespace svt { namespace table
         _rStateSet.AddState( AccessibleStateType::ENABLED );
         _rStateSet.AddState( AccessibleStateType::SENSITIVE );
         _rStateSet.AddState( AccessibleStateType::ACTIVE );
-
-        (void)_nColumnPos;
     }
 
 
-    tools::Rectangle TableControl::GetFieldCharacterBounds(sal_Int32 _nRow,sal_Int32 _nColumnPos,sal_Int32 nIndex)
+    tools::Rectangle TableControl::GetFieldCharacterBounds(sal_Int32,sal_Int32,sal_Int32 nIndex)
     {
-        (void)_nRow;
-        (void)_nColumnPos;
         return GetCharacterBounds(nIndex);
     }
 
 
-    sal_Int32 TableControl::GetFieldIndexAtPoint(sal_Int32 _nRow,sal_Int32 _nColumnPos,const Point& _rPoint)
+    sal_Int32 TableControl::GetFieldIndexAtPoint(sal_Int32,sal_Int32,const Point& _rPoint)
     {
-        (void)_nRow;
-        (void)_nColumnPos;
         return GetIndexForPoint(_rPoint);
     }
 
diff --git a/svtools/source/table/tablecontrol_impl.cxx b/svtools/source/table/tablecontrol_impl.cxx
index 08569138ccc8..c43469c40f33 100644
--- a/svtools/source/table/tablecontrol_impl.cxx
+++ b/svtools/source/table/tablecontrol_impl.cxx
@@ -112,10 +112,9 @@ namespace svt { namespace table
         {
             return false;
         }
-        virtual PColumnModel        getColumnModel( ColPos column ) override
+        virtual PColumnModel        getColumnModel( ColPos ) override
         {
             OSL_FAIL( "EmptyTableModel::getColumnModel: invalid call!" );
-            (void)column;
             return PColumnModel();
         }
         virtual PTableRenderer      getRenderer() const override
@@ -146,14 +145,8 @@ namespace svt { namespace table
         {
             return ScrollbarShowNever;
         }
-        virtual void addTableModelListener( const PTableModelListener& i_listener ) override
-        {
-            (void)i_listener;
-        }
-        virtual void removeTableModelListener( const PTableModelListener& i_listener ) override
-        {
-            (void)i_listener;
-        }
+        virtual void addTableModelListener( const PTableModelListener& ) override {}
+        virtual void removeTableModelListener( const PTableModelListener& ) override {}
         virtual ::boost::optional< ::Color > getLineColor() const override
         {
             return ::boost::optional< ::Color >();
@@ -206,18 +199,15 @@ namespace svt { namespace table
         {
             return true;
         }
-        virtual void getCellContent( ColPos const i_col, RowPos const i_row, css::uno::Any& o_cellContent ) override
+        virtual void getCellContent( ColPos const, RowPos const, css::uno::Any& o_cellContent ) override
         {
-            (void)i_row;
-            (void)i_col;
             o_cellContent.clear();
         }
         virtual void getCellToolTip( ColPos const, RowPos const, css::uno::Any& ) override
         {
         }
-        virtual Any getRowHeading( RowPos const i_rowPos ) const override
+        virtual Any getRowHeading( RowPos const ) const override
         {
-            (void)i_rowPos;
             return Any();
         }
     };
@@ -2552,9 +2542,8 @@ namespace svt { namespace table
         }
     }
 
-    void TableFunctionSet::DeselectAtPoint( const Point& rPoint )
+    void TableFunctionSet::DeselectAtPoint( const Point& )
     {
-        (void)rPoint;
         m_pTableControl->invalidateRow( m_nCurrentRow );
         m_pTableControl->markRowAsDeselected( m_nCurrentRow );
     }