RFR (S) 8208352: Merge HeapMonitorTest and HeapMonitorGCTest code

Alex Menkov alexey.menkov at oracle.com
Wed Sep 5 20:00:14 UTC 2018


+1

--alex

On 09/05/2018 12:32, serguei.spitsyn at oracle.com wrote:
> Hi Jc,
> 
> +1
> 
> Thanks,
> Serguei
> 
> 
> On 9/5/18 11:59, Chris Plummer wrote:
>> Looks good.
>>
>> Chris
>>
>> On 9/5/18 11:14 AM, JC Beyler wrote:
>>> Hi all,
>>>
>>> Two of the tests shared code and we could refactor to limit future 
>>> fixes/changes in both spots. Here is the small webrev that does that:
>>>
>>> Webrev: http://cr.openjdk.java.net/~jcbeyler/8208352/webrev.00/ 
>>> <http://cr.openjdk.java.net/%7Ejcbeyler/8208352/webrev.00/>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8208352
>>>
>>> Thank you for the future reviews,
>>> Jc
>>
>>
> 


More information about the serviceability-dev mailing list