You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by el...@apache.org on 2015/07/24 14:29:31 UTC

svn commit: r1692495 - /directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/actions/BrowserSelectionUtils.java

Author: elecharny
Date: Fri Jul 24 12:29:31 2015
New Revision: 1692495

URL: http://svn.apache.org/r1692495
Log:
o Don't use an iterator anymore, it's useless

Modified:
    directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/actions/BrowserSelectionUtils.java

Modified: directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/actions/BrowserSelectionUtils.java
URL: http://svn.apache.org/viewvc/directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/actions/BrowserSelectionUtils.java?rev=1692495&r1=1692494&r2=1692495&view=diff
==============================================================================
--- directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/actions/BrowserSelectionUtils.java (original)
+++ directory/studio/trunk/plugins/ldapbrowser.common/src/main/java/org/apache/directory/studio/ldapbrowser/common/actions/BrowserSelectionUtils.java Fri Jul 24 12:29:31 2015
@@ -386,15 +386,12 @@ public abstract class BrowserSelectionUt
         if ( selection instanceof IStructuredSelection )
         {
             IStructuredSelection structuredSelection = ( IStructuredSelection ) selection;
-            Iterator<?> it = structuredSelection.iterator();
             
-            while ( it.hasNext() )
+            for ( Object element : structuredSelection.toArray() )
             {
-                Object o = it.next();
-                
-                if ( type.isInstance( o ) )
+                if ( type.isInstance( element ) )
                 {
-                    list.add( o );
+                    list.add( element );
                 }
             }
         }