On Tue, 4 Jun 2024 07:13:39 GMT, Serguei Spitsyn <sspit...@openjdk.org> wrote:

>> Jaikiran Pai has updated the pull request incrementally with three 
>> additional commits since the last revision:
>> 
>>  - Alex suggestion - remove -XX:-CheckIntrinsics from NativeMethodPrefixApp 
>> test too
>>  - Alex's suggestion - no need to include only the bootreporter classes in 
>> boot classpath
>>  - RetransformApp test did not previously use -XX:-CheckIntrinsics
>
> test/jdk/java/lang/instrument/NativeMethodPrefixApp.java line 26:
> 
>> 24: 
>> 25: import java.io.File;
>> 26: import java.nio.file.Files;
> 
> Nit: It seems the import at line 26 is not needed.

Fixed.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19495#discussion_r1625617051

Reply via email to