Looks good to me. Thanks, Joe On 7/24/2015 1:46 AM, Daniel Fuchs wrote:
Hi,
Please find below a patch that fixes
8132256: jaxp: Investigate removal of com/sun/org/apache/bcel/internal/util/ClassPath.java https://bugs.openjdk.java.net/browse/JDK-8132256
webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8132256/webrev.00
The class com.sun.org.appache.bcel.internal.util.ClassPath does not seem to serve any useful purpose in JDK 9.
- There's only one instance created, using its default empty constructor. - There's only one method called on that instance, from SyntheticRepository.java: _path.getInputStream(className); - Because _path was created using the default (and deprecated) empty constructor, and because ClassPath is loaded from the BCL (ClassPath.class.getClassLoader() == null), then _path.getInputStream(className) will always throw an IOException.
Therefore - it seems that in the JDK - the use of com.sun.org.appache.bcel.internal.util.ClassPath is reduced to an elaborate means of throwing an IOException.
The patch above arranges for the removal of com/sun/org/apache/bcel/internal/util/ClassPath.java
best regards,
-- daniel