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/08/13 02:31:15 UTC

svn commit: r1695630 - /commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/ClassLoader.java

Author: sebb
Date: Thu Aug 13 00:31:15 2015
New Revision: 1695630

URL: http://svn.apache.org/r1695630
Log:
Don't expose arrays unnecessarily

Modified:
    commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/ClassLoader.java

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/ClassLoader.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/ClassLoader.java?rev=1695630&r1=1695629&r2=1695630&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/ClassLoader.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/ClassLoader.java Thu Aug 13 00:31:15 2015
@@ -52,12 +52,14 @@ import org.apache.commons.bcel6.classfil
  * @see JavaWrapper
  * @see ClassPath
  */
-public class ClassLoader extends java.lang.ClassLoader {
+public class ClassLoader extends java.lang.ClassLoader { // TODO rename this class to reduce confusion with parent
 
     private static final String BCEL_TOKEN = "$$BCEL$$";
-    public static final String[] DEFAULT_IGNORED_PACKAGES = {
+
+    private static final String[] DEFAULT_IGNORED_PACKAGES = {
             "java.", "javax.", "sun."
     };
+
     private final Hashtable<String, Class<?>> classes = new Hashtable<String, Class<?>>(); // Hashtable is synchronized thus thread-safe
     private final String[] ignored_packages;
     private Repository repository = SyntheticRepository.getInstance();