diff options
author | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-03-23 09:47:49 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2017-03-23 09:48:10 +0000 |
commit | ed76d1d3504c92bff6bb3e6417e4440572fcd959 (patch) | |
tree | 6d06b4a20bef5acf0c1a4118685f09acdd27fbb7 /toolkit | |
parent | a61c4ae9cef23a53ea88f957e090bd5ee9b28ca6 (diff) |
loplugins:redundantcast teach it about c-style typedef casts
Change-Id: I1ac11a2481c0f4d8be1e1fd7c7637ac0ece3d65c
Reviewed-on: https://gerrit.libreoffice.org/35558
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'toolkit')
-rw-r--r-- | toolkit/source/awt/vclxspinbutton.cxx | 8 | ||||
-rw-r--r-- | toolkit/source/awt/vclxwindow.cxx | 2 | ||||
-rw-r--r-- | toolkit/source/awt/vclxwindows.cxx | 16 | ||||
-rw-r--r-- | toolkit/source/controls/unocontrols.cxx | 8 |
4 files changed, 17 insertions, 17 deletions
diff --git a/toolkit/source/awt/vclxspinbutton.cxx b/toolkit/source/awt/vclxspinbutton.cxx index 6c82ebb39ef0..c0b598ac2319 100644 --- a/toolkit/source/awt/vclxspinbutton.cxx +++ b/toolkit/source/awt/vclxspinbutton.cxx @@ -295,19 +295,19 @@ namespace toolkit break; case BASEPROPERTY_SPINVALUE: - aReturn <<= (sal_Int32)getValue( ); + aReturn <<= getValue( ); break; case BASEPROPERTY_SPINVALUE_MIN: - aReturn <<= (sal_Int32)getMinimum( ); + aReturn <<= getMinimum( ); break; case BASEPROPERTY_SPINVALUE_MAX: - aReturn <<= (sal_Int32)getMaximum( ); + aReturn <<= getMaximum( ); break; case BASEPROPERTY_SPININCREMENT: - aReturn <<= (sal_Int32)getSpinIncrement( ); + aReturn <<= getSpinIncrement( ); break; case BASEPROPERTY_ORIENTATION: diff --git a/toolkit/source/awt/vclxwindow.cxx b/toolkit/source/awt/vclxwindow.cxx index 6171bddb78c8..1fcbc5d536b7 100644 --- a/toolkit/source/awt/vclxwindow.cxx +++ b/toolkit/source/awt/vclxwindow.cxx @@ -2112,7 +2112,7 @@ css::uno::Any VCLXWindow::getProperty( const OUString& PropertyName ) case BASEPROPERTY_REPEAT_DELAY: { sal_Int32 nButtonRepeat = GetWindow()->GetSettings().GetMouseSettings().GetButtonRepeat(); - aProp <<= (sal_Int32)nButtonRepeat; + aProp <<= nButtonRepeat; } break; diff --git a/toolkit/source/awt/vclxwindows.cxx b/toolkit/source/awt/vclxwindows.cxx index 76094cc1be3a..7d67b481f159 100644 --- a/toolkit/source/awt/vclxwindows.cxx +++ b/toolkit/source/awt/vclxwindows.cxx @@ -1560,7 +1560,7 @@ void VCLXListBox::addItems( const css::uno::Sequence< OUString>& aItems, sal_Int const OUString* pItemsEnd = aItems.getConstArray() + aItems.getLength(); while ( pItems != pItemsEnd ) { - if ( (sal_uInt16)nP == 0xFFFF ) + if ( nP == 0xFFFF ) { OSL_FAIL( "VCLXListBox::addItems: too many entries!" ); // skip remaining entries, list cannot hold them, anyway @@ -3612,37 +3612,37 @@ css::uno::Any VCLXScrollBar::getProperty( const OUString& PropertyName ) break; case BASEPROPERTY_SCROLLVALUE: { - aProp <<= (sal_Int32) getValue(); + aProp <<= getValue(); } break; case BASEPROPERTY_SCROLLVALUE_MAX: { - aProp <<= (sal_Int32) getMaximum(); + aProp <<= getMaximum(); } break; case BASEPROPERTY_SCROLLVALUE_MIN: { - aProp <<= (sal_Int32) getMinimum(); + aProp <<= getMinimum(); } break; case BASEPROPERTY_LINEINCREMENT: { - aProp <<= (sal_Int32) getLineIncrement(); + aProp <<= getLineIncrement(); } break; case BASEPROPERTY_BLOCKINCREMENT: { - aProp <<= (sal_Int32) getBlockIncrement(); + aProp <<= getBlockIncrement(); } break; case BASEPROPERTY_VISIBLESIZE: { - aProp <<= (sal_Int32) getVisibleSize(); + aProp <<= getVisibleSize(); } break; case BASEPROPERTY_ORIENTATION: { - aProp <<= (sal_Int32) getOrientation(); + aProp <<= getOrientation(); } break; case BASEPROPERTY_BACKGROUNDCOLOR: diff --git a/toolkit/source/controls/unocontrols.cxx b/toolkit/source/controls/unocontrols.cxx index a529d33cb4bf..18f6503d0483 100644 --- a/toolkit/source/controls/unocontrols.cxx +++ b/toolkit/source/controls/unocontrols.cxx @@ -443,7 +443,7 @@ void UnoEditControl::setMaxTextLen( sal_Int16 nLen ) { if ( ImplHasProperty( BASEPROPERTY_MAXTEXTLEN) ) { - ImplSetPropertyValue( GetPropertyName( BASEPROPERTY_MAXTEXTLEN ), uno::Any((sal_Int16)nLen), true ); + ImplSetPropertyValue( GetPropertyName( BASEPROPERTY_MAXTEXTLEN ), uno::Any(nLen), true ); } else { @@ -2742,7 +2742,7 @@ void UnoListBoxControl::addItems( const uno::Sequence< OUString>& aItems, sal_In OUString* pOldData = aSeq.getArray(); if ( ( nPos < 0 ) || ( nPos > nOldLen ) ) - nPos = (sal_uInt16) nOldLen; + nPos = nOldLen; sal_uInt16 n; // Items vor der Einfuege-Position @@ -2902,7 +2902,7 @@ void UnoListBoxControl::makeVisible( sal_Int16 nEntry ) void UnoListBoxControl::setDropDownLineCount( sal_Int16 nLines ) { - ImplSetPropertyValue( GetPropertyName( BASEPROPERTY_LINECOUNT ), uno::Any((sal_Int16) nLines), true ); + ImplSetPropertyValue( GetPropertyName( BASEPROPERTY_LINECOUNT ), uno::Any(nLines), true ); } sal_Int16 UnoListBoxControl::getDropDownLineCount() @@ -3363,7 +3363,7 @@ void UnoComboBoxControl::addItems( const uno::Sequence< OUString>& aItems, sal_I const OUString* pOldData = aSeq.getConstArray(); if ( ( nPos < 0 ) || ( nPos > nOldLen ) ) - nPos = (sal_uInt16) nOldLen; + nPos = nOldLen; sal_uInt16 n; // items before the insert position |