diff options
| author | Arkadiy Illarionov <qarkai@gmail.com> | 2019-08-12 23:07:08 +0300 |
|---|---|---|
| committer | Arkadiy Illarionov <qarkai@gmail.com> | 2019-08-27 12:58:33 +0200 |
| commit | 36543fc426f0358086484f9b8e439cf051d0e12b (patch) | |
| tree | dac91006fb5b9a902e2bfee05099a8ebe4c1fa71 /oox/source/ppt | |
| parent | b742b4d7f1f3bb6e1fce45ccf5d885e587e01801 (diff) | |
Simplify Sequence iterations in oox
Use range-based loops, STL and comphelper functions
Change-Id: Ic3a186e7381bd8391ab85a2602a30f06fe5db740
Reviewed-on: https://gerrit.libreoffice.org/78089
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-by: Arkadiy Illarionov <qarkai@gmail.com>
Diffstat (limited to 'oox/source/ppt')
| -rw-r--r-- | oox/source/ppt/timenode.cxx | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/oox/source/ppt/timenode.cxx b/oox/source/ppt/timenode.cxx index 6a2fcce8f469..3c59b54c9cdc 100644 --- a/oox/source/ppt/timenode.cxx +++ b/oox/source/ppt/timenode.cxx @@ -144,14 +144,12 @@ namespace oox { namespace ppt { { Reference< XAnimationNode > xEffectNode( xE2->nextElement(), UNO_QUERY_THROW ); const Sequence< NamedValue > aUserData( xEffectNode->getUserData() ); - const NamedValue* p = aUserData.getConstArray(); - sal_Int32 nLength = aUserData.getLength(); - while( nLength-- ) + for( const NamedValue& rProp : aUserData ) { - if ( p->Name == "node-type" ) + if ( rProp.Name == "node-type" ) { sal_Int16 nNodeType = 0; - p->Value >>= nNodeType; + rProp.Value >>= nNodeType; if( nNodeType != css::presentation::EffectNodeType::ON_CLICK ) { // first effect does not start on click, so correct @@ -160,7 +158,6 @@ namespace oox { namespace ppt { break; } } - p++; } } } |