On Thu, 14 May 2026 00:04:21 GMT, Anthony Scarpino <[email protected]>
wrote:
>> src/java.base/share/classes/java/security/PEMDecoder.java line 330:
>>
>>> 328: * @throws EOFException if no PEM data is found or the stream ends
>>> unexpectedly
>>> 329: * @throws IllegalArgumentException if decoding fails
>>> 330: * @throws NullPointerException when {@code is} is {@code null}
>>
>> The 2 consecutive `is` sound awkward. In fact, I've looked at other APIs
>> that take `InputStream` as an argument and most of them name it `in`.
>
> I changed it to `InputStream`. Redoing `is` for these methods and in the JEP
> is a bit much for a `{@code is} is`
That's OK. I don't think it needs any JEP change, it's just a Shift+F6 in
IntelliJ IDEA.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29640#discussion_r3242022730
