You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by mc...@apache.org on 2011/11/07 10:15:55 UTC

svn commit: r1198665 - /commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/IteratorPropertyAccessor.java

Author: mcucchiara
Date: Mon Nov  7 09:15:54 2011
New Revision: 1198665

URL: http://svn.apache.org/viewvc?rev=1198665&view=rev
Log:
OGNL-38 - Test CONSTANT.equals(variable).

Modified:
    commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/IteratorPropertyAccessor.java

Modified: commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/IteratorPropertyAccessor.java
URL: http://svn.apache.org/viewvc/commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/IteratorPropertyAccessor.java?rev=1198665&r1=1198664&r2=1198665&view=diff
==============================================================================
--- commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/IteratorPropertyAccessor.java (original)
+++ commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/IteratorPropertyAccessor.java Mon Nov  7 09:15:54 2011
@@ -41,13 +41,13 @@ public class IteratorPropertyAccessor
 
         if ( name instanceof String )
         {
-            if ( name.equals( "next" ) )
+            if ( "next".equals( name ) )
             {
                 result = iterator.next();
             }
             else
             {
-                if ( name.equals( "hasNext" ) )
+                if ( "hasNext".equals( name ) )
                 {
                     result = iterator.hasNext() ? Boolean.TRUE : Boolean.FALSE;
                 }