You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by si...@apache.org on 2011/05/22 14:23:34 UTC

svn commit: r1125939 - /incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java

Author: simonetripodi
Date: Sun May 22 12:23:34 2011
New Revision: 1125939

URL: http://svn.apache.org/viewvc?rev=1125939&view=rev
Log:
just added {} blocks

Modified:
    incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java

Modified: incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java
URL: http://svn.apache.org/viewvc/incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java?rev=1125939&r1=1125938&r2=1125939&view=diff
==============================================================================
--- incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java (original)
+++ incubator/ognl/trunk/src/main/java/org/apache/commons/ognl/ListPropertyAccessor.java Sun May 22 12:23:34 2011
@@ -74,7 +74,9 @@ public class ListPropertyAccessor
         }
 
         if ( name instanceof Number )
+        {
             return list.get( ( (Number) name ).intValue() );
+        }
 
         if ( name instanceof DynamicSubscript )
         {
@@ -82,13 +84,21 @@ public class ListPropertyAccessor
             switch ( ( (DynamicSubscript) name ).getFlag() )
             {
                 case DynamicSubscript.FIRST:
+                {
                     return len > 0 ? list.get( 0 ) : null;
+                }
                 case DynamicSubscript.MID:
+                {
                     return len > 0 ? list.get( len / 2 ) : null;
+                }
                 case DynamicSubscript.LAST:
+                {
                     return len > 0 ? list.get( len - 1 ) : null;
+                }
                 case DynamicSubscript.ALL:
+                {
                     return new ArrayList( list );
+                }
             }
         }
 
@@ -140,7 +150,9 @@ public class ListPropertyAccessor
                 case DynamicSubscript.ALL:
                 {
                     if ( !( value instanceof Collection ) )
+                    {
                         throw new OgnlException( "Value must be a collection" );
+                    }
                     list.clear();
                     list.addAll( (Collection<?>) value );
                     return;
@@ -215,7 +227,9 @@ public class ListPropertyAccessor
                 Method m = OgnlRuntime.getReadMethod( target.getClass(), indexStr );
 
                 if ( m != null )
+                {
                     return super.getSourceAccessor( context, target, index );
+                }
 
             }
             catch ( Throwable t )