From 8762cc5f41e00e6f15e67b4f691e40751588ce17 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Mon, 27 Jan 2020 08:43:10 +0100 Subject: Split independent loop vars, giving them appropriate types ...and thus avoiding an unnecessary explicit cast from sal_Int32 to sal_uInt32 Change-Id: I536fc52960259509a3b722fed9268dc6ec9e76a6 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/87482 Tested-by: Jenkins Reviewed-by: Stephan Bergmann --- toolkit/source/controls/unocontrolmodel.cxx | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'toolkit') diff --git a/toolkit/source/controls/unocontrolmodel.cxx b/toolkit/source/controls/unocontrolmodel.cxx index aa4d55237de0..a735178bdca1 100644 --- a/toolkit/source/controls/unocontrolmodel.cxx +++ b/toolkit/source/controls/unocontrolmodel.cxx @@ -785,8 +785,7 @@ void UnoControlModel::read( const css::uno::Reference< css::io::XObjectInputStre // Used for import of old parts in css::awt::FontDescriptor std::unique_ptr pFD; - sal_uInt32 i; - for ( i = 0; i < nProps; i++ ) + for ( sal_uInt32 i = 0; i < nProps; i++ ) { sal_Int32 nPropDataBeginMark = xMark->createMark(); sal_Int32 nPropDataLen = InStream->readLong(); @@ -999,7 +998,7 @@ void UnoControlModel::read( const css::uno::Reference< css::io::XObjectInputStre } if ( bInvalidEntries ) { - for ( i = 0; i < static_cast(aProps.getLength()); i++ ) + for ( sal_Int32 i = 0; i < aProps.getLength(); i++ ) { if ( aProps.getConstArray()[i].isEmpty() ) { -- cgit