Hi Sergey,

thanks for your review.

-- Igor

> On Sep 19, 2018, at 2:25 PM, Sergey Bylokhov <sergey.bylok...@oracle.com> 
> wrote:
> 
> Looks fine.
> 
> On 19/09/2018 10:18, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8210894/webrev.00/index.html
>>> 1188 lines changed: 252 ins; 591 del; 345 mod;
>> Hi all,
>> could you please review the next clean up of jdk testlibrary? now we are 
>> replacing all usages of jdk.testlibrary.Asserts w/ jdk.test.lib.Assert class 
>> and remove j.t.Asserts. While working on it, I've also replaced 
>> '(../)+lib/testlibrary/' w/  '/lib/testlibrary/' in the touched tests.
>> webrev: http://cr.openjdk.java.net/~iignatyev//8210894/webrev.00/index.html
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8210894
>> testing: tier[1-3], :jdk_svc, :jdk_security_infra and :jdk_desktop + 
>> test/jdk/java/awt tests locally on mac
>> Thanks,
>> -- Igor
> 
> 
> -- 
> Best regards, Sergey.

Reply via email to