still looking for a Reviewer.
-- Igor
> On May 31, 2017, at 2:57 PM, Igor Ignatyev <igor.ignat...@oracle.com> wrote:
>
> 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
>
