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;
/**