On Fri, 8 Jul 2022 06:12:18 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> This change uses sed to change sun.hotspot.WhiteBox to 
>> jdk.test.whitebox.Whitebox, and sun/hotspot/Whitebox similarly.  Due to 
>> indirect inclusions of some of the test libraries, changing a few wasn't a 
>> reliable option, and I need the new one for a different change I was looking 
>> at.
>> The non-sed changes are for jdk/test/whitebox/WhiteBox to add some code for 
>> GC that was only added to the sun version.
>> Also, the ClassFileInstaller has a label for sun.hotspot.Whitebox so that 
>> didn't change with the edit.
>> Tested with tiers1-6.
>
> I skimmed the diff and this seems fine.
> 
> Are we not going to remove the old WhiteBox at the same time?
> 
> Thanks.

@dholmes-ora We'll remove it with this patch JDK-8275662 and the other obsolete 
sun.hotspot test classes which will be easier to look at.
Thanks for reviewing.

-------------

PR: https://git.openjdk.org/jdk/pull/9417

Reply via email to