RFR(S): 8180195: remove jaxp testlibrary

Frank Yuan frank.yuan at oracle.com
Tue May 16 02:50:36 UTC 2017


Looks fine, although I am not a reviewer.

Thanks
Frank

> -----Original Message-----
> From: core-libs-dev [mailto:core-libs-dev-bounces at openjdk.java.net] On Behalf Of Igor Ignatyev
> Subject: RFR(S): 8180195: remove jaxp testlibrary
> 
> http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html
> > 3049 lines changed: 114 ins; 2927 del; 8 mod;
> 
> Hi all,
> 
> could you please review this small patch which removes a fork of testlibrary from jaxp repo? there were a few differences b/w the
testlibraries:
> top level testlibrary did not have CompilerUtils class, its ProcessTools did not have executeTestJava (which is basically an alias
for
> executeTestJvm) and its OutputAnalyzer did not have methods to dump stdout, stderr into specific streams.
> 
> this fix is a part of ongoing effort on merging and cleaning up our test libraries[1].
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8180195
> webrev: http://cr.openjdk.java.net/~iignatyev//8180195/webrev.00/index.html
> testing: :jaxp_all
> 
> [1] https://bugs.openjdk.java.net/browse/JDK-8075327
> 
> Thanks,
> -- Igor




More information about the core-libs-dev mailing list