On Thu, 4 Jun 2026 21:24:32 GMT, Anthony Scarpino <[email protected]> wrote:

> I need a review to add a non-public subtype to `BinaryEncodable`, causing 
> `switch` statements over `BinaryEncodable` to require either a `default` 
> label or a `case BinaryEncodable` label.
> 
> thanks
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

src/java.base/share/classes/java/security/PEMDecoder.java line 83:

> 81:  *
> 82:  * <p> The {@link #decode(String, Class)} and {@link #decode(InputStream, 
> Class)}
> 83:  * methods accept a {@code class} parameter specifying the desired {@code 
> BinaryEncodable}

Shall we put `class` into `{@code}` here? It is neither a type name (which is 
`Class`) or an identifier name (because it's a keyword).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31391#discussion_r3365444054

Reply via email to