loplugin:casttovoid: chart2

Change-Id: I63a2429860bf0ac3d80a8a43bcbde8b8f00f533f

11 files changed, 4 insertions, 24 deletions

diff --git a/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx b/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx
index 3d4f789169d7..e1d872866b21 100644
--- a/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx
+++ b/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx
@@ -1351,7 +1351,6 @@ uno::Reference< uno::XInterface > SAL_CALL ChartDocumentWrapper::createInstanceW
     const uno::Sequence< uno::Any >& Arguments )
 {
     OSL_ENSURE( Arguments.getLength(), "createInstanceWithArguments: Warning: Arguments are ignored" );
-    (void)(Arguments);
 
     return createInstance( ServiceSpecifier );
 }
diff --git a/chart2/source/controller/dialogs/DataBrowserModel.cxx b/chart2/source/controller/dialogs/DataBrowserModel.cxx
index 875137f1f576..f1d4c9dccf2d 100644
--- a/chart2/source/controller/dialogs/DataBrowserModel.cxx
+++ b/chart2/source/controller/dialogs/DataBrowserModel.cxx
@@ -686,9 +686,8 @@ bool DataBrowserModel::setCellAny( sal_Int32 nAtColumn, sal_Int32 nAtRow, const
             if( xModifiable.is() )
                 xModifiable->setModified(true);
         }
-        catch( const uno::Exception & ex )
+        catch( const uno::Exception & )
         {
-            (void)(ex);
             bResult = false;
         }
     }
@@ -899,10 +898,7 @@ void DataBrowserModel::updateFromModel()
                             Reference< beans::XPropertySet > xProp( aCooSysSeq[nCooSysIdx], uno::UNO_QUERY );
                             xProp->getPropertyValue( "SwapXAndYAxis" ) >>= bSwapXAndYAxis;
                         }
-                        catch( const beans::UnknownPropertyException & ex )
-                        {
-                            (void)ex;
-                        }
+                        catch( const beans::UnknownPropertyException & ) {}
 
                         // add ranges for error bars if present for a series
                         if( StatisticsHelper::usesErrorBarRanges( aSeries[nSeriesIdx] ))
diff --git a/chart2/source/controller/main/DrawCommandDispatch.cxx b/chart2/source/controller/main/DrawCommandDispatch.cxx
index 9c328a0887de..2fb5b4863051 100644
--- a/chart2/source/controller/main/DrawCommandDispatch.cxx
+++ b/chart2/source/controller/main/DrawCommandDispatch.cxx
@@ -279,8 +279,6 @@ FeatureState DrawCommandDispatch::getState( const OUString& rCommand )
 
 void DrawCommandDispatch::execute( const OUString& rCommand, const Sequence< beans::PropertyValue>& rArgs )
 {
-    (void)rArgs;
-
     ChartDrawMode eDrawMode = CHARTDRAW_SELECT;
     SdrObjKind eKind = OBJ_NONE;
 
diff --git a/chart2/source/controller/main/ShapeController.cxx b/chart2/source/controller/main/ShapeController.cxx
index 5404eb6df464..792e6f627687 100644
--- a/chart2/source/controller/main/ShapeController.cxx
+++ b/chart2/source/controller/main/ShapeController.cxx
@@ -141,10 +141,8 @@ FeatureState ShapeController::getState( const OUString& rCommand )
     return aReturn;
 }
 
-void ShapeController::execute( const OUString& rCommand, const Sequence< beans::PropertyValue>& rArgs )
+void ShapeController::execute( const OUString& rCommand, const Sequence< beans::PropertyValue>& )
 {
-    (void)rArgs;
-
     SupportedFeatures::const_iterator aIter = m_aSupportedFeatures.find( rCommand );
     if ( aIter != m_aSupportedFeatures.end() )
     {
diff --git a/chart2/source/model/main/Axis.cxx b/chart2/source/model/main/Axis.cxx
index 69ac5871f21e..c76ef479dd6a 100644
--- a/chart2/source/model/main/Axis.cxx
+++ b/chart2/source/model/main/Axis.cxx
@@ -332,7 +332,6 @@ void lcl_CloneSubGrids(
         ++pDestIt;
     }
     OSL_ASSERT( pDestIt == pDestEnd );
-    (void)(pDestEnd); // avoid warning
 }
 
 } // anonymous namespace
diff --git a/chart2/source/model/main/ChartModel.cxx b/chart2/source/model/main/ChartModel.cxx
index 9aaf10fdd46e..c3ebd9d794b4 100644
--- a/chart2/source/model/main/ChartModel.cxx
+++ b/chart2/source/model/main/ChartModel.cxx
@@ -1011,7 +1011,6 @@ awt::Size SAL_CALL ChartModel::getVisualAreaSize( ::sal_Int64 nAspect )
 {
     OSL_ENSURE( nAspect == embed::Aspects::MSOLE_CONTENT,
                 "No aspects other than content are supported" );
-    (void)(nAspect); // avoid warning in non-debug builds
     // other possible aspects are MSOLE_THUMBNAIL, MSOLE_ICON and MSOLE_DOCPRINT
 
     return m_aVisualAreaSize;
@@ -1021,7 +1020,6 @@ embed::VisualRepresentation SAL_CALL ChartModel::getPreferredVisualRepresentatio
 {
     OSL_ENSURE( nAspect == embed::Aspects::MSOLE_CONTENT,
                 "No aspects other than content are supported" );
-    (void)(nAspect); // avoid warning in non-debug builds
 
     embed::VisualRepresentation aResult;
 
@@ -1059,7 +1057,6 @@ embed::VisualRepresentation SAL_CALL ChartModel::getPreferredVisualRepresentatio
 {
     OSL_ENSURE( nAspect == embed::Aspects::MSOLE_CONTENT,
                 "No aspects other than content are supported" );
-    (void)(nAspect); // avoid warning in non-debug builds
     return embed::EmbedMapUnits::ONE_100TH_MM;
 }
 
@@ -1194,7 +1191,6 @@ Reference< uno::XInterface > SAL_CALL ChartModel::createInstanceWithArguments(
             const OUString& rServiceSpecifier , const Sequence< Any >& Arguments )
 {
     OSL_ENSURE( Arguments.getLength(), "createInstanceWithArguments: Warning: Arguments are ignored" );
-    (void)(Arguments); // avoid warning in non-debug builds
     return createInstance( rServiceSpecifier );
 }
 
diff --git a/chart2/source/model/main/UndoManager.cxx b/chart2/source/model/main/UndoManager.cxx
index ab6ed77744ef..7e5894b012cb 100644
--- a/chart2/source/model/main/UndoManager.cxx
+++ b/chart2/source/model/main/UndoManager.cxx
@@ -322,10 +322,9 @@ namespace chart
         return *&m_pImpl->getParent();
     }
 
-    void SAL_CALL UndoManager::setParent( const Reference< XInterface >& i_parent )
+    void SAL_CALL UndoManager::setParent( const Reference< XInterface >& )
     {
         UndoManagerMethodGuard aGuard( *m_pImpl );
-        (void)i_parent;
         throw NoSupportException( OUString(), m_pImpl->getThis() );
     }
 
diff --git a/chart2/source/tools/DiagramHelper.cxx b/chart2/source/tools/DiagramHelper.cxx
index ff2004b8c899..12f53fedc294 100644
--- a/chart2/source/tools/DiagramHelper.cxx
+++ b/chart2/source/tools/DiagramHelper.cxx
@@ -381,7 +381,6 @@ StackMode DiagramHelper::getStackModeFromChartType(
             // property is not MAYBEVOID
             bool bSuccess = ( xProp->getPropertyValue( "StackingDirection" ) >>= eCurrentDirection );
             OSL_ASSERT( bSuccess );
-            (void)(bSuccess);  // avoid warning in non-debug builds
             if( ! bDirectionInitialized )
             {
                 eCommonDirection = eCurrentDirection;
diff --git a/chart2/source/tools/LifeTime.cxx b/chart2/source/tools/LifeTime.cxx
index f24facbab19c..5df9f8f4715c 100644
--- a/chart2/source/tools/LifeTime.cxx
+++ b/chart2/source/tools/LifeTime.cxx
@@ -57,7 +57,6 @@ bool LifeTimeManager::impl_isDisposed( bool bAssert )
         if( bAssert )
         {
             OSL_FAIL( "This component is already disposed " );
-            (void)(bAssert);
         }
         return true;
     }
@@ -181,7 +180,6 @@ bool CloseableLifeTimeManager::impl_isDisposedOrClosed( bool bAssert )
         if( bAssert )
         {
             OSL_FAIL( "This object is already closed" );
-            (void)(bAssert);//avoid warnings
         }
         return true;
     }
diff --git a/chart2/source/tools/WrappedPropertySet.cxx b/chart2/source/tools/WrappedPropertySet.cxx
index f2881c3efd83..401b63e8f35b 100644
--- a/chart2/source/tools/WrappedPropertySet.cxx
+++ b/chart2/source/tools/WrappedPropertySet.cxx
@@ -244,7 +244,6 @@ void SAL_CALL WrappedPropertySet::setPropertyValues( const Sequence< OUString >&
     }
     //todo: store unknown properties elsewhere
     OSL_ENSURE(!bUnknownProperty,"unknown property");
-    (void)bUnknownProperty;
 //    if( bUnknownProperty )
 //        throw beans::UnknownPropertyException();
 }
diff --git a/chart2/source/view/main/ChartView.cxx b/chart2/source/view/main/ChartView.cxx
index 570dd20f9011..2629e998f0f9 100644
--- a/chart2/source/view/main/ChartView.cxx
+++ b/chart2/source/view/main/ChartView.cxx
@@ -3129,7 +3129,6 @@ Reference< uno::XInterface > ChartView::createInstance( const OUString& aService
 Reference< uno::XInterface > ChartView::createInstanceWithArguments( const OUString& ServiceSpecifier, const uno::Sequence< uno::Any >& Arguments )
 {
     OSL_ENSURE( Arguments.getLength(), "ChartView::createInstanceWithArguments: arguments are ignored" );
-    (void) Arguments; // avoid warning
     return createInstance( ServiceSpecifier );
 }