From bd5e62e199f30724b59f7261bd65add6375a1e16 Mon Sep 17 00:00:00 2001
From: Julien Nabet <serval2412@yahoo.fr>
Date: Sat, 30 Mar 2013 16:50:47 +0100
Subject: Prefer prefix ++/-- operators for non-primitive types

+ convert while loop into for loop

To check:
could we use std::find (find_if)?
Is it always ok to do begin()+1?
What about if we return aEnd?

Change-Id: I59107de5693ad7b37e5eef80134c34a03982151e
---
 sdext/source/minimizer/configurationaccess.cxx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'sdext')

diff --git a/sdext/source/minimizer/configurationaccess.cxx b/sdext/source/minimizer/configurationaccess.cxx
index c9ebbd6a8e36..2a3421758bc3 100644
--- a/sdext/source/minimizer/configurationaccess.cxx
+++ b/sdext/source/minimizer/configurationaccess.cxx
@@ -504,11 +504,11 @@ Sequence< PropertyValue > ConfigurationAccess::GetConfigurationSequence()
 std::vector< OptimizerSettings >::iterator ConfigurationAccess::GetOptimizerSettingsByName( const rtl::OUString& rName )
 {
     std::vector< OptimizerSettings >::iterator aIter( maSettings.begin() + 1 );
-    while ( aIter != maSettings.end() )
+    const std::vector< OptimizerSettings >::const_iterator aEnd( maSettings.end() );
+    for ( ; aIter != aEnd; ++aIter )
     {
         if ( aIter->maName == rName )
             break;
-        aIter++;
     }
     return aIter;
 }
-- 
cgit