You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by de...@apache.org on 2017/08/28 19:33:35 UTC

svn commit: r1806494 - /myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderBase.java

Author: deki
Date: Mon Aug 28 19:33:34 2017
New Revision: 1806494

URL: http://svn.apache.org/viewvc?rev=1806494&view=rev
Log:
MYFACES-4135 simplified code

Modified:
    myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderBase.java

Modified: myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderBase.java
URL: http://svn.apache.org/viewvc/myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderBase.java?rev=1806494&r1=1806493&r2=1806494&view=diff
==============================================================================
--- myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderBase.java (original)
+++ myfaces/core/branches/2.3.x/impl/src/main/java/org/apache/myfaces/el/unified/ResolverBuilderBase.java Mon Aug 28 19:33:34 2017
@@ -193,14 +193,7 @@ public class ResolverBuilderBase
             try
             {
                 // filter the resolvers
-                for (Iterator<ELResolver> iter = resolvers.iterator(); iter.hasNext();)
-                {
-                    ELResolver elResolver = iter.next();
-                    if (predicate.test(elResolver) == false)
-                    {
-                        iter.remove();
-                    }
-                }
+                resolvers.removeIf(elResolver -> !predicate.test(elResolver));
 
                 if (log.isLoggable(Level.INFO))
                 {