Hi Mandy, 8180793[1], which moved j.t.l.wrappers.* to j.t.lib package, has been propagated to jdk10/jdk10. I have updated the patch accordingly -- http://cr.openjdk.java.net/~iignatyev//8180386/webrev.01/index.html could you please take a look?
[1] https://bugs.openjdk.java.net/browse/JDK-8180793 Thanks, -- Igor > On May 18, 2017, at 2:49 PM, Igor Ignatyev <igor.ignat...@oracle.com> wrote: > >> >> On May 18, 2017, at 2:41 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: >> >> >>> On May 18, 2017, at 2:37 PM, Igor Ignatyev <igor.ignat...@oracle.com> wrote: >>> >>> >>>> On May 18, 2017, at 2:34 PM, Mandy Chung <mandy.ch...@oracle.com> wrote: >>>> This comment is not related to the change. The package name >>>> `jdk.test.lib.wrappers` is odd. TimeLimitedRunner and InfiniteLoop classes >>>> could simply be in the jdk.test.lib package or jdk.test.lib.util package. >>>> >>>> Mandy >>>> >>> >>> agree, I'll file an RFE to find a better package for TimeLimitedRunner and >>> InfiniteLoop classes. >> >> Why not doing the rename with this patch? > > there are hotspot tests which use jdk.test.lib.wrappers.TimeLimitedRunner and > InfiniteLoop, so it will require changes in hotspot repo. I'd prefer to have > removing classes from jdk testlibrary and renaming classes in top level > testlibrary as separate patches for clarity purposes. > > -- Igor