commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject svn commit: r1209104 - in /commons/proper/jexl/branches/2.0-API-COMPAT: ./ src/main/java/org/apache/commons/jexl2/ src/main/java/org/apache/commons/jexl2/introspection/
Date Thu, 01 Dec 2011 14:33:53 GMT
Author: sebb
Date: Thu Dec  1 14:33:50 2011
New Revision: 1209104

URL: http://svn.apache.org/viewvc?rev=1209104&view=rev
Log:
Synch. code changes from main 2.0 branch (mainly @since markers)

Modified:
    commons/proper/jexl/branches/2.0-API-COMPAT/RELEASE-NOTES.txt
    commons/proper/jexl/branches/2.0-API-COMPAT/pom.xml
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/DebugInfo.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/ExpressionImpl.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/Interpreter.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlArithmetic.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlEngine.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlException.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlInfo.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/UnifiedJEXL.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
    commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/UberspectImpl.java

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/RELEASE-NOTES.txt
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/RELEASE-NOTES.txt?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/RELEASE-NOTES.txt (original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/RELEASE-NOTES.txt Thu Dec  1 14:33:50 2011
@@ -24,7 +24,7 @@ $Id$
 INTRODUCTION:
 =============
 
-JEXL is a library intended to facilitate the implementation of dynamic and scripting features
in
+The Apache Commons JEXL library facilitates the implementation of dynamic and scripting features
in
 applications and frameworks written in Java.
 
 JEXL implements an Expression Language based on some extensions to the JSTL Expression Language
supporting most of the

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/pom.xml
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/pom.xml?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/pom.xml (original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/pom.xml Thu Dec  1 14:33:50 2011
@@ -107,7 +107,7 @@
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>
-            <version>3.8.2</version>
+            <version>4.10</version>
             <scope>test</scope>
         </dependency>
         <!-- For JSR-223 API -->
@@ -227,7 +227,7 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-checkstyle-plugin</artifactId>
-                <version>2.5</version>
+                <version>2.7</version>
                 <configuration>
                     <configLocation>${basedir}/src/main/config/checkstyle.xml</configLocation>
                     <excludes>org/apache/commons/jexl2/parser/*.java</excludes>
@@ -255,7 +255,7 @@
             <plugin>
                 <groupId>org.codehaus.mojo</groupId>
                 <artifactId>findbugs-maven-plugin</artifactId>
-                <version>2.3.1</version>
+                <version>2.3.2</version>
                 <configuration>
                     <excludeFilterFile>${basedir}/src/main/config/findbugs-exclude-filter.xml</excludeFilterFile>
                     <xmlOutput>true</xmlOutput>

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/DebugInfo.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/DebugInfo.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/DebugInfo.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/DebugInfo.java
Thu Dec  1 14:33:50 2011
@@ -63,7 +63,9 @@ public class DebugInfo implements JexlIn
         return toString();
     }
     
-    /** {@inheritDoc} */
+    /** {@inheritDoc} 
+     * @since 2.1 
+     */
     public DebugInfo debugInfo() {
         return this;
     }

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/ExpressionImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/ExpressionImpl.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/ExpressionImpl.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/ExpressionImpl.java
Thu Dec  1 14:33:50 2011
@@ -109,6 +109,7 @@ public class ExpressionImpl implements E
 
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public Object execute(JexlContext context, Object... args) {
         Interpreter interpreter = jexl.createInterpreter(context);
@@ -118,6 +119,7 @@ public class ExpressionImpl implements E
 
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public String[] getParameters() {
         return script.getParameters();
@@ -125,6 +127,7 @@ public class ExpressionImpl implements E
 
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public String[] getLocalVariables() {
         return script.getLocalVariables();
@@ -132,6 +135,7 @@ public class ExpressionImpl implements E
 
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public Set<List<String>> getVariables() {
         return jexl.getVariables(this);
@@ -139,6 +143,7 @@ public class ExpressionImpl implements E
 
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public Callable<Object> callable(JexlContext context) {
         return callable(context, (Object[]) null);
@@ -146,6 +151,7 @@ public class ExpressionImpl implements E
 
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public Callable<Object> callable(JexlContext context, Object... args) {
         final Interpreter interpreter = jexl.createInterpreter(context);

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/Interpreter.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/Interpreter.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/Interpreter.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/Interpreter.java
Thu Dec  1 14:33:50 2011
@@ -113,10 +113,21 @@ public class Interpreter implements Pars
     protected final boolean cache;
     /** Registers or arguments. */
     protected Object[] registers = null;
-    /** Parameter names if any. */
-    protected String[] parameters = null; // TODO does not appear to be read
-    /** Cancellation support. */
-    protected volatile boolean cancelled = false; // TODO could this be private?
+    /**
+     * Parameter names if any.
+     * Intended for use in debugging; not currently used externally.
+     * @since 2.1 
+     */
+    @SuppressWarnings("unused")
+    private String[] parameters = null;
+
+    /** 
+     * Cancellation support.
+     * @see #isCancelled()
+     * @since 2.1
+     */
+    private volatile boolean cancelled = false;
+
     /** Empty parameters for method matching. */
     protected static final Object[] EMPTY_PARAMS = new Object[0];
 
@@ -137,6 +148,7 @@ public class Interpreter implements Pars
      * @param aContext the context to evaluate expression
      * @param strictFlag whether this interpreter runs in strict mode
      * @param silentFlag whether this interpreter runs in silent mode
+     * @since 2.1
      */
     public Interpreter(JexlEngine jexl, JexlContext aContext, boolean strictFlag, boolean
silentFlag) {
         this.logger = jexl.logger;
@@ -153,6 +165,7 @@ public class Interpreter implements Pars
     /**
      * Copy constructor.
      * @param base the base to copy
+     * @since 2.1
      */
     protected Interpreter(Interpreter base) {
         this.logger = base.logger;
@@ -170,7 +183,9 @@ public class Interpreter implements Pars
      * Sets whether this interpreter considers unknown variables, methods and constructors
as errors.
      * @param flag true for strict, false for lenient
      * @deprecated Do not use; will be removed in a later version
+     * @since 2.1 
      */
+    // TODO why add a method and then deprecate it?
     @Deprecated
     public void setStrict(boolean flag) {
         this.strict = flag;
@@ -189,6 +204,7 @@ public class Interpreter implements Pars
     /**
      * Checks whether this interpreter considers unknown variables, methods and constructors
as errors.
      * @return true if strict, false otherwise
+     * @since 2.1
      */
     public boolean isStrict() {
         return this.strict;
@@ -233,6 +249,7 @@ public class Interpreter implements Pars
     /**
      * Gets the context.
      * @return the {@link JexlContext} used for evaluation.
+     * @since 2.1
      */
     protected JexlContext getContext() {
         return context;
@@ -266,6 +283,7 @@ public class Interpreter implements Pars
     /**
      * Sets this interpreter parameters and arguments.
      * @param frame the calling frame
+     * @since 2.1
      */
     protected void setFrame(JexlEngine.Frame frame) {
         if (frame != null) {
@@ -1329,13 +1347,19 @@ public class Interpreter implements Pars
         return result;
     }
 
-    /** {@inheritDoc} */
+    /** 
+     * {@inheritDoc}
+     * @since 2.1 
+     */
     public Object visit(ASTReferenceExpression node, Object data) {
         ASTArrayAccess upper = node;
         return visit(upper, data);
     }
 
-    /** {@inheritDoc} */
+    /** 
+     * {@inheritDoc}
+     * @since 2.1 
+     */
     public Object visit(ASTReturnStatement node, Object data) {
         Object val = node.jjtGetChild(0).jjtAccept(this, data);
         throw new JexlException.Return(node, null, val);

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlArithmetic.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlArithmetic.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlArithmetic.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlArithmetic.java
Thu Dec  1 14:33:50 2011
@@ -52,14 +52,23 @@ public class JexlArithmetic {
     protected static final BigInteger BIGI_LONG_MAX_VALUE = BigInteger.valueOf(Long.MAX_VALUE);
     /** Long.MIN_VALUE as BigInteger. */
     protected static final BigInteger BIGI_LONG_MIN_VALUE = BigInteger.valueOf(Long.MIN_VALUE);
-    /** Default BigDecimal scale. */
+    /** 
+     * Default BigDecimal scale. 
+     * @since 2.1 
+     */
     protected static final int BIGD_SCALE = -1;
     /** Whether this JexlArithmetic instance behaves in strict or lenient mode. 
      */
     private volatile boolean strict;
-    /** The big decimal math context. */
+    /** 
+     * The big decimal math context.
+     * @since 2.1 
+     */
     protected final MathContext mathContext;
-    /** The big decimal scale. */
+    /** 
+     * The big decimal scale.
+     * @since 2.1
+     */
     protected final int mathScale;
 
     /**
@@ -82,6 +91,21 @@ public class JexlArithmetic {
         this.mathContext = bigdContext;
         this.mathScale = bigdScale;
     }
+    
+    
+    /**
+     * Sets whether this JexlArithmetic instance triggers errors during evaluation when
+     * null is used as an operand.
+     * <p>This method is thread safe, however using it whilst an expression is 
+     * currently being evaluated is not recommended.</p>
+     * @see JexlEngine#setLenient
+     * @see JexlEngine#setSilent
+     * @see JexlEngine#setDebug
+     * @param flag true means no JexlException will occur, false allows them
+     */
+    void setLenient(boolean flag) {
+        this.strict = !flag;
+    }
 
     /**
      * Checks whether this JexlArithmetic instance triggers errors during evaluation
@@ -102,7 +126,7 @@ public class JexlArithmetic {
     }
 
     /**
-     * The BigDecimal scale used for comparison and coericion operations.
+     * The BigDecimal scale used for comparison and coercion operations.
      * @return the scale
      * @since 2.1
      */
@@ -238,6 +262,7 @@ public class JexlArithmetic {
      * @param rhs the right hand side operand that lead to the bigd result
      * @param bigd the BigDecimal to narrow
      * @return an Integer or Long if narrowing is possible, the original BigInteger otherwise
+     * @since 2.1
      */
     protected Number narrowBigDecimal(Object lhs, Object rhs, BigDecimal bigd) {
         if (isNumberable(lhs) || isNumberable(rhs)) {
@@ -540,6 +565,7 @@ public class JexlArithmetic {
      * Negates a value (unary minus for numbers).
      * @param val the value to negate
      * @return the negated value
+     * @since 2.1
      */
     public Object negate(Object val) {
         if (val instanceof Integer) {
@@ -1037,6 +1063,7 @@ public class JexlArithmetic {
      * @param narrow the target narrow class
      * @param source the orginal source class
      * @return true if attempt to narrow source to target is accepted
+     * @since 2.1
      */
     protected boolean narrowAccept(Class<?> narrow, Class<?> source) {
         return narrow == null || narrow.equals(source);
@@ -1047,6 +1074,7 @@ public class JexlArithmetic {
      * @param original the original number
      * @param narrow the attempted target class
      * @return  the narrowed number or the source if no narrowing was possible
+     * @since 2.1
      */
     protected Number narrowNumber(Number original, Class<?> narrow) {
         if (original == null) {
@@ -1110,18 +1138,4 @@ public class JexlArithmetic {
         }
         return result;
     }
-
-    /**
-     * Sets whether this JexlArithmetic instance triggers errors during evaluation when
-     * null is used as an operand.
-     * <p>This method is thread safe, however using it whilst an expression is 
-     * currently being evaluated is not recommended.</p>
-     * @see JexlEngine#setLenient
-     * @see JexlEngine#setSilent
-     * @see JexlEngine#setDebug
-     * @param flag true means no JexlException will occur, false allows them
-     */
-    void setLenient(boolean flag) {
-        this.strict = !flag;
-    }
 }
\ No newline at end of file

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlEngine.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlEngine.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlEngine.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlEngine.java
Thu Dec  1 14:33:50 2011
@@ -230,6 +230,7 @@ public class JexlEngine {
     /**
      * Gets this engine underlying arithmetic.
      * @return the arithmetic
+     * @since 2.1
      */
     public JexlArithmetic getArithmetic() {
         return arithmetic;
@@ -312,6 +313,7 @@ public class JexlEngine {
      * If a single JexlEngine is shared between threads, 
      * it should only be called when the engine is not being used.</p>
      * @param flag true for strict, false for lenient
+     * @since 2.1
      */
     public final void setStrict(boolean flag) {
         setLenient(!flag);
@@ -321,6 +323,7 @@ public class JexlEngine {
      * Checks whether this engine behaves in strict or lenient mode.
      * Equivalent to !isLenient().
      * @return true for strict, false for lenient
+     * @since 2.1
      */
     public final boolean isStrict() {
         return !isLenient();
@@ -496,6 +499,7 @@ public class JexlEngine {
      * @param names the script parameter names
      * @return A {@link Script} which can be executed using a {@link JexlContext}.
      * @throws JexlException if there is a problem parsing the script.
+     * @since 2.1
      */
     public Script createScript(String scriptText, JexlInfo info, String[] names) {
         if (scriptText == null) {
@@ -787,6 +791,7 @@ public class JexlEngine {
      * @param strictFlag whether the interpreter runs in strict mode
      * @param silentFlag whether the interpreter runs in silent mode
      * @return an Interpreter
+     * @since 2.1
      */
     protected Interpreter createInterpreter(JexlContext context, boolean strictFlag, boolean
silentFlag) {
         return new Interpreter(this, context == null ? EMPTY_CONTEXT : context, strictFlag,
silentFlag);
@@ -887,6 +892,7 @@ public class JexlEngine {
 
     /**
      * Clears the expression cache.
+     * @since 2.1
      */
     public void clearCache() {
         synchronized (parser) {
@@ -901,6 +907,7 @@ public class JexlEngine {
      * @param script the script
      * @return the set of variables, each as a list of strings (ant-ish variables use more
than 1 string)
      *         or the empty set if no variables are used
+     * @since 2.1
      */
     public Set<List<String>> getVariables(Script script) {
         if (script instanceof ExpressionImpl) {
@@ -917,6 +924,7 @@ public class JexlEngine {
      * @param node the node
      * @param refs the set of variable being filled
      * @param ref the current variable being filled
+     * @since 2.1
      */
     protected void getVariables(JexlNode node, Set<List<String>> refs, List<String>
ref) {
         boolean array = node instanceof ASTArrayAccess;
@@ -977,6 +985,7 @@ public class JexlEngine {
      * Gets the array of parameters from a script.
      * @param script the script
      * @return the parameters which may be empty (but not null) if no parameters were defined
+     * @since 2.1
      */
     protected String[] getParameters(Script script) {
         if (script instanceof ExpressionImpl) {
@@ -990,6 +999,7 @@ public class JexlEngine {
      * Gets the array of local variable from a script.
      * @param script the script
      * @return the local variables array which may be empty (but not null) if no local variables
were defined
+     * @since 2.1
      */
     protected String[] getLocalVariables(Script script) {
         if (script instanceof ExpressionImpl) {
@@ -1001,18 +1011,19 @@ public class JexlEngine {
 
     /**
      * A script scope, stores the declaration of parameters and local variables.
+     * @since 2.1
      */
     public static final class Scope {
         /**
          * The number of parameters.
          */
-        protected final int parms;
+        private final int parms;
         /**
          * The map of named registers aka script parameters.
          * Each parameter is associated to a register and is materialized as an offset in
the registers array used
          * during evaluation.
          */
-        protected Map<String, Integer> namedRegisters = null;
+        private Map<String, Integer> namedRegisters = null;
 
         /**
          * Creates a new scope with a list of parameters.
@@ -1160,12 +1171,13 @@ public class JexlEngine {
 
     /**
      * A call frame, created from a scope, stores the arguments and local variables as "registers".
+     * @since 2.1
      */
     public static final class Frame {
         /** Registers or arguments. */
-        protected Object[] registers = null;
+        private Object[] registers = null;
         /** Parameter and argument names if any. */
-        protected String[] parameters = null;
+        private String[] parameters = null;
         
         /**
          * Creates a new frame.

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlException.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlException.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlException.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlException.java
Thu Dec  1 14:33:50 2011
@@ -103,6 +103,7 @@ public class JexlException extends Runti
     /**
      * Accesses detailed message.
      * @return  the message
+     * @since 2.1
      */
     protected String detailedMessage() {
         return super.getMessage();
@@ -113,6 +114,7 @@ public class JexlException extends Runti
      * @param prefix the prefix to the message
      * @param expr the expression in error
      * @return the formatted message
+     * @since 2.1
      */
     protected String parserError(String prefix, String expr) {
         int begin = info.debugInfo().getColumn();

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlInfo.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlInfo.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlInfo.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/JexlInfo.java
Thu Dec  1 14:33:50 2011
@@ -29,6 +29,7 @@ public interface JexlInfo {
     /**
      * Gets the underlying debugging information.
      * @return a debug info instance
+     * @since 2.1
      */
     DebugInfo debugInfo();
 }

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/UnifiedJEXL.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/UnifiedJEXL.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/UnifiedJEXL.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/UnifiedJEXL.java
Thu Dec  1 14:33:50 2011
@@ -211,6 +211,7 @@ public final class UnifiedJEXL {
 
     /**
      * Clears the cache.
+     * @since 2.1
      */
     public void clearCache() {
         synchronized (cache) {
@@ -313,6 +314,7 @@ public final class UnifiedJEXL {
          * are written in 'dot' or 'bracketed' notation. (a.b is equivalent to a['b']).</p>
          * @return the set of variables, each as a list of strings (ant-ish variables use
more than 1 string)
          *         or the empty set if no variables are used
+         * @since 2.1
          */
         public Set<List<String>> getVariables() {
             return Collections.emptySet();
@@ -321,6 +323,7 @@ public final class UnifiedJEXL {
         /**
          * Fills up the list of variables accessed by this expression.
          * @param refs the set of variable being filled
+         * @since 2.1
          */
         protected void getVariables(Set<List<String>> refs) {
             // nothing to do
@@ -1340,6 +1343,7 @@ public final class UnifiedJEXL {
      * @param sequence the sequence
      * @param pattern the pattern to match at start of sequence
      * @return the first position after end of pattern if it matches, -1 otherwise
+     * @since 2.1
      */
     protected int startsWith(CharSequence sequence, CharSequence pattern) {
         int s = 0;

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
Thu Dec  1 14:33:50 2011
@@ -197,7 +197,7 @@ public final class Sandbox {
      */
     public static final class WhiteSet extends Names {
         /** The map of controlled names and aliases. */
-        protected Map<String, String> names = null;
+        private Map<String, String> names = null;
 
         @Override
         public boolean add(String name) {
@@ -230,7 +230,7 @@ public final class Sandbox {
      */
     public static final class BlackSet extends Names {
         /** The set of controlled names. */
-        protected Set<String> names = null;
+        private Set<String> names = null;
 
         @Override
         public boolean add(String name) {

Modified: commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/UberspectImpl.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/UberspectImpl.java?rev=1209104&r1=1209103&r2=1209104&view=diff
==============================================================================
--- commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/UberspectImpl.java
(original)
+++ commons/proper/jexl/branches/2.0-API-COMPAT/src/main/java/org/apache/commons/jexl2/introspection/UberspectImpl.java
Thu Dec  1 14:33:50 2011
@@ -61,6 +61,7 @@ public class UberspectImpl extends Intro
     /**
      * Resets this Uberspect class loader.
      * @param cloader the class loader to use
+     * @since 2.1
      */
     public void setLoader(ClassLoader cloader) {
         base().setLoader(cloader);
@@ -117,6 +118,7 @@ public class UberspectImpl extends Intro
     
     /**
      * {@inheritDoc}
+     * @since 2.1
      */
     public JexlMethod getConstructorMethod(Object ctorHandle, Object[] args, JexlInfo info)
{
         final Constructor<?> ctor = getConstructor(ctorHandle, args);
@@ -181,6 +183,7 @@ public class UberspectImpl extends Intro
      * @param object the object
      * @param name the container name
      * @return a JexlPropertyGet is successfull, null otherwise
+     * @since 2.1
      */
     protected JexlPropertyGet getIndexedGet(Object object, String name) {
         if (object != null && name != null) {



Mime
View raw message