RFR(XS): 8157468: gc/testlibrary contains a lot of duplicated code

Igor Ignatyev igor.ignatyev at oracle.com
Fri Sep 2 13:35:37 UTC 2016


Hi Kirill,

looks good to me.

Thanks,
— Igor

> On Aug 31, 2016, at 10:00 AM, Kirill Zhaldybin <kirill.zhaldybin at oracle.com> wrote:
> 
> Dmitry,
> 
> Thank you for review!
> 
> Regards, Kirill
> 
> On 08/30/2016 11:48 PM, Dmitry Fazunenko wrote:
>> Looks good.
>> 
>> On 30.08.2016 22:40, Kirill Zhaldybin wrote:
>>> Dear all,
>>> 
>>> Could you please review this fix for 8157468?
>>> 
>>> Duplicated code was deleted, minor refactoring done, deleted not needed import.
>>> 
>>> WebRev: http://cr.openjdk.java.net/~kzhaldyb/webrevs/JDK-8157468/webrev.00/
>>> CR: https://bugs.openjdk.java.net/browse/JDK-8157468
>>> 
>>> Thank you.
>>> 
>>> Regards, Kirill
>> 
> 



More information about the hotspot-gc-dev mailing list