You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2015/12/09 15:51:56 UTC

svn commit: r1718874 - in /commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser: ASTJexlScript.java JexlParser.java

Author: sebb
Date: Wed Dec  9 14:51:56 2015
New Revision: 1718874

URL: http://svn.apache.org/viewvc?rev=1718874&view=rev
Log:
Don't expose variables unnecessarily

Modified:
    commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/ASTJexlScript.java
    commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/JexlParser.java

Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/ASTJexlScript.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/ASTJexlScript.java?rev=1718874&r1=1718873&r2=1718874&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/ASTJexlScript.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/ASTJexlScript.java Wed Dec  9 14:51:56 2015
@@ -24,9 +24,9 @@ import java.util.Map;
  */
 public class ASTJexlScript extends JexlNode {
     /** The script scope. */
-    protected Scope scope = null;
+    private Scope scope = null;
     /** The pragmas. */
-    protected Map<String, Object> pragmas = null;
+    Map<String, Object> pragmas = null;
 
     public ASTJexlScript(int id) {
         super(id);

Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/JexlParser.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/JexlParser.java?rev=1718874&r1=1718873&r2=1718874&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/JexlParser.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/JexlParser.java Wed Dec  9 14:51:56 2015
@@ -33,20 +33,20 @@ import java.util.Stack;
  */
 public abstract class JexlParser extends StringParser {
     /** Whether the parser will allow user-named registers (aka #0 syntax). */
-    protected boolean ALLOW_REGISTERS = false;
+    boolean ALLOW_REGISTERS = false;
     /** The source being processed. */
-    protected String source = null;
+    String source = null;
     /**
      * The map of named registers aka script parameters.
      * <p>Each parameter is associated to a register and is materialized
      * as an offset in the registers array used during evaluation.</p>
      */
-    protected Scope frame = null;
-    protected Stack<Scope> frames = new Stack<Scope>();
+    Scope frame = null;
+    Stack<Scope> frames = new Stack<Scope>();
     /**
      * The list of pragma declarations.
      */
-    protected Map<String, Object> pragmas = null;
+    Map<String, Object> pragmas = null;
 
 
     /**