diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-05 13:03:43 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-05-08 08:29:03 +0200 |
commit | 185ed3ddb8c01ee4465ce559e37113824f57b5c7 (patch) | |
tree | 596455ca4b9dc85666efbf06a1e1e0a3eec3ee2d /svtools | |
parent | d33e262a244f351febc9dbe605b05f76cb834eeb (diff) |
teach loplugin:constantparam about simple constructor calls
Change-Id: I7d2a28ab5951fbdb5a427c84e9ac4c1e32ecf9f9
Reviewed-on: https://gerrit.libreoffice.org/37280
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'svtools')
-rw-r--r-- | svtools/source/contnr/svlbitm.cxx | 1 | ||||
-rw-r--r-- | svtools/source/control/toolbarmenu.cxx | 4 | ||||
-rw-r--r-- | svtools/source/table/tablecontrol.cxx | 2 | ||||
-rw-r--r-- | svtools/source/table/tablecontrol_impl.cxx | 4 | ||||
-rw-r--r-- | svtools/source/table/tablecontrol_impl.hxx | 2 |
5 files changed, 6 insertions, 7 deletions
diff --git a/svtools/source/contnr/svlbitm.cxx b/svtools/source/contnr/svlbitm.cxx index 7e1cb420a7d7..f520b747a8c1 100644 --- a/svtools/source/contnr/svlbitm.cxx +++ b/svtools/source/contnr/svlbitm.cxx @@ -351,7 +351,6 @@ void SvLBoxButton::ImplAdjustBoxSize(Size& io_rSize, ControlType i_eType, vcl::R aCtrlRegion, nState, aControlValue, - OUString(), aNativeBounds, aNativeContent ); if( bNativeOK ) diff --git a/svtools/source/control/toolbarmenu.cxx b/svtools/source/control/toolbarmenu.cxx index 92d23b77f701..17c666288fc6 100644 --- a/svtools/source/control/toolbarmenu.cxx +++ b/svtools/source/control/toolbarmenu.cxx @@ -525,7 +525,7 @@ static long ImplGetNativeCheckAndRadioSize(vcl::RenderContext& rRenderContext, l if (rRenderContext.IsNativeControlSupported(ControlType::MenuPopup, ControlPart::MenuItemCheckMark)) { if (rRenderContext.GetNativeControlRegion(ControlType::MenuPopup, ControlPart::MenuItemCheckMark, - aCtrlRegion, ControlState::ENABLED, aVal, OUString(), + aCtrlRegion, ControlState::ENABLED, aVal, aNativeBounds, aNativeContent) ) { @@ -536,7 +536,7 @@ static long ImplGetNativeCheckAndRadioSize(vcl::RenderContext& rRenderContext, l if (rRenderContext.IsNativeControlSupported(ControlType::MenuPopup, ControlPart::MenuItemRadioMark)) { if (rRenderContext.GetNativeControlRegion(ControlType::MenuPopup, ControlPart::MenuItemRadioMark, - aCtrlRegion, ControlState::ENABLED, aVal, OUString(), + aCtrlRegion, ControlState::ENABLED, aVal, aNativeBounds, aNativeContent) ) { diff --git a/svtools/source/table/tablecontrol.cxx b/svtools/source/table/tablecontrol.cxx index bd18666c6bd6..7fca9670e48f 100644 --- a/svtools/source/table/tablecontrol.cxx +++ b/svtools/source/table/tablecontrol.cxx @@ -652,7 +652,7 @@ namespace svt { namespace table if ( m_pImpl->isAccessibleAlive() ) { - m_pImpl->commitAccessibleEvent( AccessibleEventId::SELECTION_CHANGED, Any(), Any() ); + m_pImpl->commitAccessibleEvent( AccessibleEventId::SELECTION_CHANGED ); m_pImpl->commitTableEvent( AccessibleEventId::ACTIVE_DESCENDANT_CHANGED, Any(), Any() ); // TODO: why do we notify this when the *selection* changed? Shouldn't we find a better place for this, diff --git a/svtools/source/table/tablecontrol_impl.cxx b/svtools/source/table/tablecontrol_impl.cxx index 6ac56256baee..4fc4f5d5dfb1 100644 --- a/svtools/source/table/tablecontrol_impl.cxx +++ b/svtools/source/table/tablecontrol_impl.cxx @@ -2318,9 +2318,9 @@ namespace svt { namespace table } - void TableControl_Impl::commitAccessibleEvent( sal_Int16 const i_eventID, const Any& i_newValue, const Any& i_oldValue ) + void TableControl_Impl::commitAccessibleEvent( sal_Int16 const i_eventID ) { - impl_commitAccessibleEvent( i_eventID, i_newValue, i_oldValue ); + impl_commitAccessibleEvent( i_eventID, Any(), Any() ); } diff --git a/svtools/source/table/tablecontrol_impl.hxx b/svtools/source/table/tablecontrol_impl.hxx index e03942aec4f8..1cb24ae7af49 100644 --- a/svtools/source/table/tablecontrol_impl.hxx +++ b/svtools/source/table/tablecontrol_impl.hxx @@ -238,7 +238,7 @@ namespace svt { namespace table */ bool markAllRowsAsSelected(); - void commitAccessibleEvent( sal_Int16 const i_eventID, const css::uno::Any& i_newValue, const css::uno::Any& i_oldValue ); + void commitAccessibleEvent( sal_Int16 const i_eventID ); void commitCellEvent( sal_Int16 const i_eventID, const css::uno::Any& i_newValue, const css::uno::Any& i_oldValue ); void commitTableEvent( sal_Int16 const i_eventID, const css::uno::Any& i_newValue, const css::uno::Any& i_oldValue ); |