Author: grobmeier
Date: Fri Oct 14 07:40:32 2011
New Revision: 1183232

URL: http://svn.apache.org/viewvc?rev=1183232&view=rev
Log:
some checkstyle

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

Modified: 
commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/ASTAdd.java
URL: 
http://svn.apache.org/viewvc/commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/ASTAdd.java?rev=1183232&r1=1183231&r2=1183232&view=diff
==============================================================================
--- commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/ASTAdd.java 
(original)
+++ commons/proper/ognl/trunk/src/main/java/org/apache/commons/ognl/ASTAdd.java 
Fri Oct 14 07:40:32 2011
@@ -1,3 +1,4 @@
+package org.apache.commons.ognl;
 /*
  * $Id$
  * Licensed to the Apache Software Foundation (ASF) under one
@@ -17,7 +18,6 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-package org.apache.commons.ognl;
 
 import org.apache.commons.ognl.enhance.ExpressionCompiler;
 
@@ -51,8 +51,9 @@ class ASTAdd
     {
         Object result = _children[0].getValue( context, source );
 
-        for ( int i = 1; i < _children.length; ++i )
+        for ( int i = 1; i < _children.length; ++i ) {
             result = OgnlOps.add( result, _children[i].getValue( context, 
source ) );
+        }
 
         return result;
     }
@@ -64,48 +65,74 @@ class ASTAdd
 
     boolean isWider( NodeType type, NodeType lastType )
     {
-        if ( lastType == null )
+        if ( lastType == null ) 
+        {
             return true;
+        }
 
         // System.out.println("checking isWider(" + type.getGetterClass() + " 
, " + lastType.getGetterClass() + ")");
 
-        if ( String.class.isAssignableFrom( lastType.getGetterClass() ) )
+        if ( String.class.isAssignableFrom( lastType.getGetterClass() ) ) 
+        {
             return false;
+        }
 
-        if ( String.class.isAssignableFrom( type.getGetterClass() ) )
+        if ( String.class.isAssignableFrom( type.getGetterClass() ) ) 
+        {
             return true;
+        }
 
-        if ( _parent != null && String.class.isAssignableFrom( 
type.getGetterClass() ) )
+        if ( _parent != null && String.class.isAssignableFrom( 
type.getGetterClass() ) ) 
+        {
             return true;
+        }
 
-        if ( String.class.isAssignableFrom( lastType.getGetterClass() ) && 
Object.class == type.getGetterClass() )
+        if ( String.class.isAssignableFrom( lastType.getGetterClass() ) && 
Object.class == type.getGetterClass() ) 
+        {
             return false;
+        }
 
-        if ( _parent != null && String.class.isAssignableFrom( 
lastType.getGetterClass() ) )
+        if ( _parent != null && String.class.isAssignableFrom( 
lastType.getGetterClass() ) ) 
+        {
             return false;
+        }
         else if ( _parent == null && String.class.isAssignableFrom( 
lastType.getGetterClass() ) )
-            return true;
+        {
+               return true;
+        }
         else if ( _parent == null && String.class.isAssignableFrom( 
type.getGetterClass() ) )
-            return false;
+        {
+               return false;
+        }
 
         if ( BigDecimal.class.isAssignableFrom( type.getGetterClass() )
             || BigInteger.class.isAssignableFrom( type.getGetterClass() ) )
-            return true;
+        {
+               return true;
+        }
 
         if ( BigDecimal.class.isAssignableFrom( lastType.getGetterClass() )
             || BigInteger.class.isAssignableFrom( lastType.getGetterClass() ) )
-            return false;
+        {
+               return false;
+        }
 
         if ( Double.class.isAssignableFrom( type.getGetterClass() ) )
-            return true;
+        {
+               return true;
+        }
 
         if ( Integer.class.isAssignableFrom( type.getGetterClass() )
             && Double.class.isAssignableFrom( lastType.getGetterClass() ) )
-            return false;
+        {
+               return false;
+        }
 
         if ( Float.class.isAssignableFrom( type.getGetterClass() )
             && Integer.class.isAssignableFrom( lastType.getGetterClass() ) )
-            return true;
+        {
+               return true;
+        }
 
         return true;
     }
@@ -303,7 +330,7 @@ class ASTAdd
         }
     }
     
-    public <R,P> R accept(NodeVisitor<? extends R, ? super P> visitor, P data) 
+    public <R,P> R accept( NodeVisitor<? extends R, ? super P> visitor, P data 
) 
     {
         return visitor.visit(this, data);
     }


Reply via email to