Re: RFR 8197418: Move java/util/RandomAccess/ tests into OpenJDK

2018-04-11 Thread Chris Yin
Thank you, Paul

Regards,
Chris

> On 12 Apr 2018, at 2:10 AM, Paul Sandoz  wrote:
> 
> Hi Chris,
> 
> +1, and thanks for sharing code coverage results.
> 
> Paul.
> 
> 
>> On Apr 10, 2018, at 11:33 PM, Chris Yin  wrote:
>> 
>> Please review the change to move java/util/RandomAccess/Basic.java test into 
>> OpenJDK, it increases code coverage of open jdk_util test group per JCov 
>> run, the main improvement is on java.util.Collections with %method up 1% and 
>> %block up 2%, thanks
>> 
>> bug: https://bugs.openjdk.java.net/browse/JDK-8197418 
>> 
>> webrev: http://cr.openjdk.java.net/~xyin/8197418/webrev.00/ 
>> 
>> 
>> Regards,
>> Chris
> 



Re: RFR 8197418: Move java/util/RandomAccess/ tests into OpenJDK

2018-04-11 Thread Paul Sandoz
Hi Chris,

+1, and thanks for sharing code coverage results.

Paul.


> On Apr 10, 2018, at 11:33 PM, Chris Yin  wrote:
> 
> Please review the change to move java/util/RandomAccess/Basic.java test into 
> OpenJDK, it increases code coverage of open jdk_util test group per JCov run, 
> the main improvement is on java.util.Collections with %method up 1% and 
> %block up 2%, thanks
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8197418 
> 
> webrev: http://cr.openjdk.java.net/~xyin/8197418/webrev.00/ 
> 
> 
> Regards,
> Chris