[commons-jexl] 05/19: There does not seem to be any reason for these fields to be protected rather than private. Can always increase access later; much more difficult to decrease access.

[henrib]

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 828454d885012ce0bf445d21be9a0dbed373a19d
Author: Sebastian Bazley <s...@apache.org>
AuthorDate: Thu Dec 1 12:33:02 2011 +0000

    There does not seem to be any reason for these fields to be protected 
rather than private.
    Can always increase access later; much more difficult to decrease access.
    
    git-svn-id: 
https://svn-us.apache.org/repos/asf/commons/proper/jexl/branches/2.0@1209050 
13f79535-47bb-0310-9956-ffa450edef68
---
 src/main/java/org/apache/commons/jexl2/JexlEngine.java            | 8 ++++----
 src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/main/java/org/apache/commons/jexl2/JexlEngine.java 
b/src/main/java/org/apache/commons/jexl2/JexlEngine.java
index 6f523dc..bcbd460 100644
--- a/src/main/java/org/apache/commons/jexl2/JexlEngine.java
+++ b/src/main/java/org/apache/commons/jexl2/JexlEngine.java
@@ -996,13 +996,13 @@ public class JexlEngine {
         /**
          * 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.
@@ -1154,9 +1154,9 @@ public class JexlEngine {
      */
     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.
diff --git a/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java 
b/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
index 5013a05..36c0411 100644
--- a/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
+++ b/src/main/java/org/apache/commons/jexl2/introspection/Sandbox.java
@@ -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) {