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/16 09:51:45 UTC
svn commit: r1696114 - in /commons/proper/bcel/trunk/src: changes/changes.xml
main/java/org/apache/commons/bcel6/util/JavaWrapper.java
Author: sebb
Date: Sun Aug 16 07:51:45 2015
New Revision: 1696114
URL: http://svn.apache.org/r1696114
Log:
BCEL-110 Problem with JAXB if the bcel classloader is used; remove the broken ClassLoader class
Modified:
commons/proper/bcel/trunk/src/changes/changes.xml
commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/JavaWrapper.java
Modified: commons/proper/bcel/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/changes/changes.xml?rev=1696114&r1=1696113&r2=1696114&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/changes/changes.xml (original)
+++ commons/proper/bcel/trunk/src/changes/changes.xml Sun Aug 16 07:51:45 2015
@@ -64,7 +64,7 @@ The <action> type attribute can be add,u
<body>
<release version="6.0" date="TBA" description="Major release with Java 7 and 8 support">
<action issue="BCEL-231" type="remove">Remove deprecated methods and classes</action>
- <action issue="BCEL-110" type="remove">Problem with JAXB if the bcel classloader is used</action>
+ <action issue="BCEL-110" type="remove">Problem with JAXB if the bcel classloader is used; remove the broken ClassLoader class</action>
<action issue="BCEL-201" type="update">modify several toString methods to make output similar to "javap"</action>
<action issue="BCEL-205" type="update">add javadoc comments to LineNumber.java and LineNumberTable.java</action>
<action issue="BCEL-208" type="fix">Need to check for an empty InstructionList</action>
Modified: commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/JavaWrapper.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/JavaWrapper.java?rev=1696114&r1=1696113&r2=1696114&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/JavaWrapper.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/commons/bcel6/util/JavaWrapper.java Sun Aug 16 07:51:45 2015
@@ -28,8 +28,7 @@ import java.lang.reflect.Modifier;
*
* <pre>java org.apache.commons.bcel6.util.JavaWrapper <real.class.name> [arguments]</pre>
*
- * <p>To use your own class loader you can set the "bcel.classloader" system property
- * which defaults to "org.apache.commons.bcel6.util.ClassLoader", e.g., with:</p>
+ * <p>To use your own class loader you can set the "bcel.classloader" system property<p>
* <pre>java org.apache.commons.bcel6.util.JavaWrapper -Dbcel.classloader=foo.MyLoader <real.class.name> [arguments]</pre>
*
* @version $Id$
@@ -43,7 +42,7 @@ public class JavaWrapper {
private static java.lang.ClassLoader getClassLoader() {
String s = System.getProperty("bcel.classloader");
if ((s == null) || "".equals(s)) {
- s = "org.apache.commons.bcel6.util.ClassLoader";
+ throw new IllegalArgumentException("The property 'bcel.classloader' must be defined");
}
try {
return (java.lang.ClassLoader) Class.forName(s).newInstance();