diff options
| author | Arkadiy Illarionov <qarkai@gmail.com> | 2019-08-03 22:59:04 +0300 |
|---|---|---|
| committer | Arkadiy Illarionov <qarkai@gmail.com> | 2019-08-09 16:58:06 +0200 |
| commit | fca94779872b8ba0b0583d0b7068f1a46beb88c5 (patch) | |
| tree | c3dd200a046d3c6e54cba570f7661b9c1068ed1b /sc/source/ui/vba/vbaeventshelper.cxx | |
| parent | d05eb786d408af883500404229b89ffa0d7cde16 (diff) | |
Simplify Sequence iterations in sc
Use range-based loops, STL and comphelper functions
Change-Id: I047fb2e6ec9591166339b9748c5013a32185f14b
Reviewed-on: https://gerrit.libreoffice.org/76912
Tested-by: Jenkins
Reviewed-by: Arkadiy Illarionov <qarkai@gmail.com>
Diffstat (limited to 'sc/source/ui/vba/vbaeventshelper.cxx')
| -rw-r--r-- | sc/source/ui/vba/vbaeventshelper.cxx | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/sc/source/ui/vba/vbaeventshelper.cxx b/sc/source/ui/vba/vbaeventshelper.cxx index 251843efaea6..51897fa525c5 100644 --- a/sc/source/ui/vba/vbaeventshelper.cxx +++ b/sc/source/ui/vba/vbaeventshelper.cxx @@ -376,12 +376,11 @@ void SAL_CALL ScVbaEventListener::changesOccurred( const util::ChangesEvent& rEv } ScRangeList aRangeList; - for( sal_Int32 nIndex = 0; nIndex < nCount; ++nIndex ) + for( const util::ElementChange& rChange : rEvent.Changes ) { - aChange = rEvent.Changes[ nIndex ]; - aChange.Accessor >>= sOperation; + rChange.Accessor >>= sOperation; uno::Reference< table::XCellRange > xRangeObj; - aChange.ReplacedElement >>= xRangeObj; + rChange.ReplacedElement >>= xRangeObj; if( xRangeObj.is() && sOperation.equalsIgnoreAsciiCase("cell-change") ) { uno::Reference< sheet::XCellRangeAddressable > xCellRangeAddressable( xRangeObj, uno::UNO_QUERY ); @@ -773,8 +772,7 @@ uno::Sequence< uno::Any > ScVbaEventsHelper::implBuildArgumentList( const EventH sal_Int32 nLength = aVbaArgs.getLength(); uno::Sequence< uno::Any > aVbaArgs2( nLength + 1 ); aVbaArgs2[ 0 ] = createWorksheet( rArgs, 0 ); - for( sal_Int32 nIndex = 0; nIndex < nLength; ++nIndex ) - aVbaArgs2[ nIndex + 1 ] = aVbaArgs[ nIndex ]; + std::copy_n(aVbaArgs.begin(), nLength, std::next(aVbaArgs2.begin())); aVbaArgs = aVbaArgs2; } |