You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by pr...@apache.org on 2003/09/04 02:36:15 UTC

cvs commit: jakarta-commons/jexl/src/java/org/apache/commons/jexl/parser ASTMethod.java

proyal      2003/09/03 17:36:15

  Modified:    jexl/src/java/org/apache/commons/jexl/parser ASTMethod.java
  Log:
  20729: Patch from Mark Wilkinson to remove unneeded e.printStackTrace()
  
  Revision  Changes    Path
  1.5       +4 -10     jakarta-commons/jexl/src/java/org/apache/commons/jexl/parser/ASTMethod.java
  
  Index: ASTMethod.java
  ===================================================================
  RCS file: /home/cvs/jakarta-commons/jexl/src/java/org/apache/commons/jexl/parser/ASTMethod.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- ASTMethod.java	25 Jun 2003 11:17:00 -0000	1.4
  +++ ASTMethod.java	4 Sep 2003 00:36:15 -0000	1.5
  @@ -30,7 +30,7 @@
        *  returns the value of itself applied to the object.
        *   We assume that an identifier can be gotten via a get(String)
        */
  -    public Object execute(Object obj, JexlContext jc) 
  +    public Object execute(Object obj, JexlContext jc)
           throws Exception
       {
           String methodName = ((ASTIdentifier)jjtGetChild(0)).val;
  @@ -49,13 +49,13 @@
               {
                   params[i] = ( (SimpleNode) jjtGetChild(i+1)).value(jc);
               }
  -    
  +
               VelMethod vm = Introspector.getUberspect().getMethod(obj, methodName,
                   params, new Info("",1,1));
  -    
  +
               if (vm == null)
                   return null;
  -    
  +
               return vm.invoke(obj, params);
           }
           catch(InvocationTargetException e)
  @@ -69,11 +69,5 @@
   
               throw e;
           }
  -        catch(Exception e)
  -        {
  -            System.out.println("ASTMethod : "+ e);
  -            e.printStackTrace();;
  -            throw e;
  -         }
       }
   }