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/10 18:09:32 UTC

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

Author: sebb
Date: Mon Aug 10 16:09:32 2015
New Revision: 1695125

URL: http://svn.apache.org/r1695125
Log:
Use constant for special BCEL token string; replace the magic number (the token length)

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=1695125&r1=1695124&r2=1695125&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 Mon Aug 10 16:09:32 2015
@@ -54,6 +54,7 @@ import org.apache.commons.bcel6.classfil
  */
 public class ClassLoader extends java.lang.ClassLoader {
 
+    private static final String BCEL_TOKEN = "$$BCEL$$";
     public static final String[] DEFAULT_IGNORED_PACKAGES = {
             "java.", "javax.", "sun."
     };
@@ -115,7 +116,7 @@ public class ClassLoader extends java.la
                 JavaClass clazz = null;
                 /* Third try: Special request?
                  */
-                if (class_name.contains("$$BCEL$$")) {
+                if (class_name.contains(BCEL_TOKEN)) {
                     clazz = createClass(class_name);
                 } else { // Fourth try: Load classes via repository
                     if ((clazz = repository.loadClass(class_name)) != null) {
@@ -163,8 +164,8 @@ public class ClassLoader extends java.la
      * @param class_name compressed byte code with "$$BCEL$$" in it
      */
     protected JavaClass createClass( String class_name ) {
-        int index = class_name.indexOf("$$BCEL$$");
-        String real_name = class_name.substring(index + 8);
+        int index = class_name.indexOf(BCEL_TOKEN);
+        String real_name = class_name.substring(index + BCEL_TOKEN.length());
         JavaClass clazz = null;
         try {
             byte[] bytes = Utility.decode(real_name, true);