RfR - 8130058: jaxp: Investigate removal of com/sun/org/apache/xalan/internal/xslt/Process.java

[Daniel Fuchs]

Hi,

Please find below an updated webrev:

http://cr.openjdk.java.net/~dfuchs/webrev_8130058/webrev.01/

Instead of removing the CLITest.java - I copied
Process.java into the test hierarchy, renamed it to
ProcessXSLT.java, modified it so that it no longer uses
internal APIs, and changed the CLITest.java to use that.

The only adherence is with
com.sun.org.apache.xml.internal.utils.DefaultErrorHandler
that the new ProcessXSLT class tries to instantiate
through reflection - as that error handler has access to
jaxp internals and can give a better error diagnostic.

However - the ProcessXSLT will use its own dummy ErrorHandler
if it can't managed to instantiate the internal class.

This way - we can keep the test :-)

best regards,

-- daniel

On 29/07/15 17:02, Daniel Fuchs wrote:
> Hi,
>
> Please find below a patch that removes a bunch of unused files
> in jdk9/dev/jaxp:
>
> https://bugs.openjdk.java.net/browse/JDK-8130058
> 8130058: jaxp: Investigate removal of
>           com/sun/org/apache/xalan/internal/xslt/Process.java
>
> http://cr.openjdk.java.net/~dfuchs/webrev_8130058/webrev.00/
>
> best regards,
>
> -- daniel