diff options
author | Arkadiy Illarionov <qarkai@gmail.com> | 2018-12-03 00:52:17 +0300 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-12-03 08:14:15 +0100 |
commit | 056845403c387f1fe286f9df13f73fa2fc6d0014 (patch) | |
tree | 34fcb1196b3a1f863bc87915355cb24143cfa9bc /sdext/source/minimizer/fileopendialog.cxx | |
parent | 83a72f46d34fde7a119b00fcc0a7bf58dbe0076a (diff) |
Simplify containers iterations in sdext
Use range-based loop or replace with STL functions
Change-Id: I760c1aaeae9afc99effee6a2645bb77439260ddf
Reviewed-on: https://gerrit.libreoffice.org/64435
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sdext/source/minimizer/fileopendialog.cxx')
-rw-r--r-- | sdext/source/minimizer/fileopendialog.cxx | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/sdext/source/minimizer/fileopendialog.cxx b/sdext/source/minimizer/fileopendialog.cxx index 13728bc5cba9..472951469571 100644 --- a/sdext/source/minimizer/fileopendialog.cxx +++ b/sdext/source/minimizer/fileopendialog.cxx @@ -118,12 +118,12 @@ FileOpenDialog::FileOpenDialog( const Reference< XComponentContext >& rxContext Reference< XNameAccess > xTypes( rxContext->getServiceManager()->createInstanceWithContext( "com.sun.star.document.TypeDetection", rxContext ), UNO_QUERY_THROW ); - for( std::vector< FilterEntry >::const_iterator aIter(aFilterEntryList.begin()), aEnd(aFilterEntryList.end()); aIter != aEnd; ++aIter ) + for( const auto& rFilterEntry : aFilterEntryList ) { Sequence< PropertyValue > aTypeProperties; try { - if ( xTypes->getByName( aIter->maType ) >>= aTypeProperties ) + if ( xTypes->getByName( rFilterEntry.maType ) >>= aTypeProperties ) { Sequence< OUString > aExtensions; for ( int i = 0; i < aTypeProperties.getLength(); i++ ) @@ -139,10 +139,10 @@ FileOpenDialog::FileOpenDialog( const Reference< XComponentContext >& rxContext // The filter title must be formed in the same way it is // currently done in the internal implementation: OUString aTitle( - aIter->maUIName + " (." + aExtensions[0] + ")"); + rFilterEntry.maUIName + " (." + aExtensions[0] + ")"); OUString aFilter("*." + aExtensions[0]); mxFilePicker->appendFilter(aTitle, aFilter); - if ( aIter->maFlags & 0x100 ) + if ( rFilterEntry.maFlags & 0x100 ) mxFilePicker->setCurrentFilter(aTitle); } } @@ -173,14 +173,10 @@ OUString FileOpenDialog::getFilterName() const OUString aFilterName; Reference< XFilterManager > xFilterManager( mxFilePicker, UNO_QUERY_THROW ); OUString aUIName( xFilterManager->getCurrentFilter() ); - for( std::vector< FilterEntry >::const_iterator aIter(aFilterEntryList.begin()), aEnd(aFilterEntryList.end()); aIter != aEnd; ++aIter ) - { - if ( aIter->maUIName == aUIName ) - { - aFilterName = aIter->maFilterEntryName; - break; - } - } + auto aIter = std::find_if(aFilterEntryList.begin(), aFilterEntryList.end(), + [&aUIName](const FilterEntry& rFilterEntry) { return rFilterEntry.maUIName == aUIName; }); + if (aIter != aFilterEntryList.end()) + aFilterName = aIter->maFilterEntryName; return aFilterName; }; |