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);


Reply via email to