On Mon, 2 Oct 2023 14:40:49 GMT, Maurizio Cimadamore <mcimadam...@openjdk.org> 
wrote:

>> Adam Sotona has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   fixed javadoc typo
>
> test/langtools/tools/javac/7166455/CheckACC_STRICTFlagOnclinitTest.java line 
> 107:
> 
>> 105:     }
>> 106: 
>> 107: // this version of the code can be used when ClassFile API in not in a 
>> preview
> 
> Do you want to keep these commented lines? I'm fine (as it seems like you 
> saved the code that was there before, for future reference). I'm just asking 
> as this is a biggie PR, in case something went amiss.

That said, if tests with such usages are not many, we could also consider 
adding enable-preview to the tests, and just use the new API.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/15706#discussion_r1342789740

Reply via email to