You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by he...@apache.org on 2021/06/03 15:41:49 UTC

[commons-jexl] 06/19: No need for cancelled and parameters to be non-private

This is an automated email from the ASF dual-hosted git repository.

henrib pushed a commit to tag 2.1
in repository https://gitbox.apache.org/repos/asf/commons-jexl.git

commit 1cb1ca57702c44fd9d738ace25ac976cd67bb09d
Author: Sebastian Bazley <se...@apache.org>
AuthorDate: Thu Dec 1 13:45:11 2011 +0000

    No need for cancelled and parameters to be non-private
    
    git-svn-id: https://svn-us.apache.org/repos/asf/commons/proper/jexl/branches/2.0@1209075 13f79535-47bb-0310-9956-ffa450edef68
---
 src/main/java/org/apache/commons/jexl2/Interpreter.java | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/apache/commons/jexl2/Interpreter.java b/src/main/java/org/apache/commons/jexl2/Interpreter.java
index 5cf475b..da0cfd4 100644
--- a/src/main/java/org/apache/commons/jexl2/Interpreter.java
+++ b/src/main/java/org/apache/commons/jexl2/Interpreter.java
@@ -113,14 +113,19 @@ public class Interpreter implements ParserVisitor {
     protected Object[] registers = null;
     /**
      * Parameter names if any.
+     * Intended for use in debugging; not currently used externally.
      * @since 2.1 
      */
-    protected String[] parameters = null;
+    @SuppressWarnings("unused")
+    private String[] parameters = null;
+
     /** 
      * Cancellation support.
+     * @see #isCancelled()
      * @since 2.1
      */
-    protected volatile boolean cancelled = false;
+    private volatile boolean cancelled = false;
+
     /** Empty parameters for method matching. */
     protected static final Object[] EMPTY_PARAMS = new Object[0];