On Tue, 28 Oct 2025 10:08:55 GMT, Alan Bateman <[email protected]> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Alan's suggestion - rename "is" to "in"
>
> src/java.base/share/classes/jdk/internal/module/ModulePatcher.java line 470:
>
>> 468: @Override
>> 469: public ByteBuffer getByteBuffer() throws IOException {
>> 470: try (InputStream is = getInputStream()) {
>
> Can you rename "is" to "in", only because "is" isn't a great name.
Done, in the updated version of this PR.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28014#discussion_r2468881063
