RFR(S): 8180195: remove jaxp testlibrary

huizhe wang huizhe.wang at oracle.com
Tue May 16 03:21:09 UTC 2017


+1. Thanks Frank for checking.

-Joe

On 5/15/2017 7:50 PM, Frank Yuan wrote:
> 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