loplugin:casttovoid: reportdesign

Change-Id: Ie7d26ac2369d8555dbd17925158d693bb8a37d7a

9 files changed, 4 insertions, 18 deletions

diff --git a/reportdesign/source/core/api/ReportEngineJFree.cxx b/reportdesign/source/core/api/ReportEngineJFree.cxx
index 9c04a42119f7..8661cf2b94bc 100644
--- a/reportdesign/source/core/api/ReportEngineJFree.cxx
+++ b/reportdesign/source/core/api/ReportEngineJFree.cxx
@@ -252,9 +252,8 @@ OUString OReportEngineJFree::getNewOutputName()
             if ( sOutputName.isEmpty() )
                 throw lang::IllegalArgumentException();
         }
-        catch(const uno::Exception& e)
+        catch(const uno::Exception&)
         {
-            (void)e; // helper to know what e contains
             throw;
         }
     }
diff --git a/reportdesign/source/core/misc/conditionalexpression.cxx b/reportdesign/source/core/misc/conditionalexpression.cxx
index c51a757b3f35..c9b2f2dae29b 100644
--- a/reportdesign/source/core/misc/conditionalexpression.cxx
+++ b/reportdesign/source/core/misc/conditionalexpression.cxx
@@ -65,11 +65,6 @@ namespace rptui
 
     bool ConditionalExpression::matchExpression( const OUString& _rExpression, const OUString& _rFieldDataSource, OUString& _out_rLHS, OUString& _out_rRHS ) const
     {
-        (void)_rExpression;
-        (void)_rFieldDataSource;
-        (void)_out_rLHS;
-        (void)_out_rRHS;
-
         // if we had regular expression, the matching would be pretty easy ...
         // just replace $1 and $2 in the pattern with (.*), and then get them with \1 resp. \2.
         // Unfortunately, we don't have such a regexp engine ...
diff --git a/reportdesign/source/core/sdr/PropertyForward.cxx b/reportdesign/source/core/sdr/PropertyForward.cxx
index f2abe0dc7ae0..d65386302c23 100644
--- a/reportdesign/source/core/sdr/PropertyForward.cxx
+++ b/reportdesign/source/core/sdr/PropertyForward.cxx
@@ -76,10 +76,9 @@ OPropertyMediator::OPropertyMediator(const Reference< XPropertySet>& _xSource
             }
             startListening();
         }
-        catch(Exception& e)
+        catch(Exception&)
         {
             DBG_UNHANDLED_EXCEPTION();
-            (void)e;
         }
     }
     osl_atomic_decrement(&m_refCount);
diff --git a/reportdesign/source/core/sdr/RptPage.cxx b/reportdesign/source/core/sdr/RptPage.cxx
index 4e254a62b63f..8fd6efa22b6a 100644
--- a/reportdesign/source/core/sdr/RptPage.cxx
+++ b/reportdesign/source/core/sdr/RptPage.cxx
@@ -155,8 +155,7 @@ void OReportPage::removeTempObject(SdrObject *_pToRemoveObj)
             SdrObject *aObj = GetObj(i);
             if (aObj && aObj == _pToRemoveObj)
             {
-                SdrObject* pObject = RemoveObject(i);
-                (void)pObject;
+                (void) RemoveObject(i);
                 break;
             }
         }
diff --git a/reportdesign/source/filter/xml/xmlfilter.cxx b/reportdesign/source/filter/xml/xmlfilter.cxx
index 7a5bfa5cbd9b..f691cf7bbbac 100644
--- a/reportdesign/source/filter/xml/xmlfilter.cxx
+++ b/reportdesign/source/filter/xml/xmlfilter.cxx
@@ -150,7 +150,6 @@ ErrCode ReadThroughComponent(
     }
     catch (const SAXParseException& r)
     {
-        (void)r;
         SAL_WARN( "reportdesign", "SAX parse exception caught while importing: "
                     << r.Message << " "
                     << r.LineNumber << ',' << r.ColumnNumber );
diff --git a/reportdesign/source/ui/dlg/AddField.cxx b/reportdesign/source/ui/dlg/AddField.cxx
index 1693963389b5..4d89af8070e7 100644
--- a/reportdesign/source/ui/dlg/AddField.cxx
+++ b/reportdesign/source/ui/dlg/AddField.cxx
@@ -268,7 +268,6 @@ bool OAddFieldWindow::PreNotify( NotifyEvent& _rNEvt )
 void OAddFieldWindow::_propertyChanged( const beans::PropertyChangeEvent& _evt )
 {
     OSL_ENSURE( _evt.Source == m_xRowSet, "OAddFieldWindow::_propertyChanged: where did this come from?" );
-    (void)_evt;
     Update();
 }
 
diff --git a/reportdesign/source/ui/report/ReportControllerObserver.cxx b/reportdesign/source/ui/report/ReportControllerObserver.cxx
index c2cf2b261a31..6ceacbe92010 100644
--- a/reportdesign/source/ui/report/ReportControllerObserver.cxx
+++ b/reportdesign/source/ui/report/ReportControllerObserver.cxx
@@ -125,7 +125,6 @@ public:
     // XEventListener
     void SAL_CALL OXReportControllerObserver::disposing(const lang::EventObject& e)
     {
-        (void) e;
         // check if it's an object we have cached information about
         uno::Reference< beans::XPropertySet > xSourceSet(e.Source, uno::UNO_QUERY);
         if ( xSourceSet.is() )
@@ -147,7 +146,6 @@ public:
     // XPropertyChangeListener
     void SAL_CALL OXReportControllerObserver::propertyChange(const beans::PropertyChangeEvent& _rEvent)
     {
-        (void) _rEvent;
         ::osl::ClearableMutexGuard aGuard( m_pImpl->m_aMutex );
 
         if ( m_pImpl->m_nLocks != 0 )
diff --git a/reportdesign/source/ui/report/ReportWindow.cxx b/reportdesign/source/ui/report/ReportWindow.cxx
index aaf608f335bf..8ac0e90fa598 100644
--- a/reportdesign/source/ui/report/ReportWindow.cxx
+++ b/reportdesign/source/ui/report/ReportWindow.cxx
@@ -429,9 +429,8 @@ sal_uInt16 OReportWindow::getZoomFactor(SvxZoomType _eType) const
     return nZoom;
 }
 
-void OReportWindow::_propertyChanged(const beans::PropertyChangeEvent& _rEvent)
+void OReportWindow::_propertyChanged(const beans::PropertyChangeEvent&)
 {
-    (void)_rEvent;
     Resize();
     m_aViewsWindow->Resize();
     Invalidate(InvalidateFlags::Transparent);
diff --git a/reportdesign/source/ui/report/dlgedfunc.cxx b/reportdesign/source/ui/report/dlgedfunc.cxx
index 2511747f11d6..62b8fe3fef35 100644
--- a/reportdesign/source/ui/report/dlgedfunc.cxx
+++ b/reportdesign/source/ui/report/dlgedfunc.cxx
@@ -622,7 +622,6 @@ bool DlgEdFunc::setMovementPointer(const MouseEvent& rMEvt)
     else
     {
         bool bCtrlKey = rMEvt.IsMod1();
-        (void)bCtrlKey;
         if (bCtrlKey)
         {
             m_pParent->SetPointer( Pointer(PointerStyle::MoveDataLink ));