You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by di...@apache.org on 2006/04/26 07:11:30 UTC

svn commit: r397092 - in /jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl: Expression.java ExpressionFactory.java ExpressionImpl.java JexlContext.java JexlExprResolver.java JexlHelper.java Script.java

Author: dion
Date: Tue Apr 25 22:11:28 2006
New Revision: 397092

URL: http://svn.apache.org/viewcvs?rev=397092&view=rev
Log:
Checkstyle

Modified:
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Expression.java
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionFactory.java
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionImpl.java
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlContext.java
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlExprResolver.java
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlHelper.java
    jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Script.java

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Expression.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Expression.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Expression.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Expression.java Tue Apr 25 22:11:28 2006
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002,2004 The Apache Software Foundation.
+ * Copyright 2002-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionFactory.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionFactory.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionFactory.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionFactory.java Tue Apr 25 22:11:28 2006
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002,2004 The Apache Software Foundation.
+ * Copyright 2002-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionImpl.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionImpl.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionImpl.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/ExpressionImpl.java Tue Apr 25 22:11:28 2006
@@ -1,5 +1,5 @@
 /*
- * Copyright 2002,2004 The Apache Software Foundation.
+ * Copyright 2002-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -142,7 +142,8 @@
      * allows addition of a resolver to allow custom interdiction of expression
      * evaluation.
      *
-     * @param resolver resolver to be called before a Jexl expression is evaluated.
+     * @param resolver resolver to be called before a Jexl expression 
+     *      is evaluated.
      */
     public void addPreResolver(JexlExprResolver resolver) {
         if (preResolvers == null) {

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlContext.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlContext.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlContext.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlContext.java Tue Apr 25 22:11:28 2006
@@ -1,12 +1,12 @@
 /*
- * Copyright 2002,2004 The Apache Software Foundation.
+ * Copyright 2002-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlExprResolver.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlExprResolver.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlExprResolver.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlExprResolver.java Tue Apr 25 22:11:28 2006
@@ -1,12 +1,12 @@
 /*
- * Copyright 2002,2004 The Apache Software Foundation.
+ * Copyright 2002-2006 The Apache Software Foundation.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlHelper.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlHelper.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlHelper.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/JexlHelper.java Tue Apr 25 22:11:28 2006
@@ -1,12 +1,12 @@
 /*
- * Copyright 2002,2004 The Apache Software Foundation.
- * 
+ * Copyright 2002-2006 The Apache Software Foundation.
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -28,12 +28,12 @@
  *  @author <a href="mailto:geirm@apache.org">Geir Magnusson Jr.</a>
  *  @version $Id$
  */
-public class JexlHelper
-{
+public class JexlHelper {
+    /** singleton instance. */
     protected static JexlHelper helper = new JexlHelper();
 
-    protected static JexlHelper getInstance()
-    {
+    /** @return the single instance. */
+    protected static JexlHelper getInstance() {
         return helper;
     }
 
@@ -41,18 +41,17 @@
      * Returns a new {@link JexlContext}.
      * @return a new JexlContext
      */
-    public static JexlContext createContext()
-    {
+    public static JexlContext createContext() {
         return getInstance().newContext();
     }
 
     /**
-     * Creates and returns a new {@link JexlContext}.  The current implementation
-     * creates a new instance of {@link HashMapContext}.
+     * Creates and returns a new {@link JexlContext}.  
+     * The current implementation creates a new instance of 
+     * {@link HashMapContext}.
      * @return a new JexlContext
      */
-    protected JexlContext newContext()
-    {
+    protected JexlContext newContext() {
         return new HashMapContext();
     }
 }

Modified: jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Script.java
URL: http://svn.apache.org/viewcvs/jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Script.java?rev=397092&r1=397091&r2=397092&view=diff
==============================================================================
--- jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Script.java (original)
+++ jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl/Script.java Tue Apr 25 22:11:28 2006
@@ -1,12 +1,12 @@
 /*
  * Copyright 2002-2006 The Apache Software Foundation.
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -32,7 +32,9 @@
      * supplied {@link JexlContext}. 
      * 
      * @param context A JexlContext containing variables.
-     * @return The result of this script, usually the result of the last statement.
+     * @return The result of this script, usually the result of 
+     *      the last statement.
+     * @throws Exception on any script parse or execution error.
      */
     Object execute(JexlContext context) throws Exception;
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Re: svn commit: r397092 - in /jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl: Expression.java ExpressionFactory.java ExpressionImpl.java JexlContext.java JexlExprResolver.java JexlHelper.java Script.java

Posted by Dion Gillard <di...@gmail.com>.
Go for it.

On 4/26/06, Rahul Akolkar <ra...@gmail.com> wrote:
>
> On 4/26/06, dion@apache.org <di...@apache.org> wrote:
> > Author: dion
> > Date: Tue Apr 25 22:11:28 2006
> > New Revision: 397092
> >
> > URL: http://svn.apache.org/viewcvs?rev=397092&view=rev
> > Log:
> > Checkstyle
> >
> <snip/>
>
> I can do a package or two, if you want. Leaving jexl.parser aside, do
> you want to split the remaining? Or are you done already? ;-)
>
> -Rahul
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
>
>


--
http://www.multitask.com.au/people/dion/
Chuck Norris sleeps with a night light. Not because Chuck Norris is afraid
of the dark, but because the dark is afraid of Chuck Norris

Re: svn commit: r397092 - in /jakarta/commons/proper/jexl/trunk/src/java/org/apache/commons/jexl: Expression.java ExpressionFactory.java ExpressionImpl.java JexlContext.java JexlExprResolver.java JexlHelper.java Script.java

Posted by Rahul Akolkar <ra...@gmail.com>.
On 4/26/06, dion@apache.org <di...@apache.org> wrote:
> Author: dion
> Date: Tue Apr 25 22:11:28 2006
> New Revision: 397092
>
> URL: http://svn.apache.org/viewcvs?rev=397092&view=rev
> Log:
> Checkstyle
>
<snip/>

I can do a package or two, if you want. Leaving jexl.parser aside, do
you want to split the remaining? Or are you done already? ;-)

-Rahul

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org