RFR: JDK-8346465 : Add a check in setData() to restrict the update of Built-In ICC_Profiles [v5]

Alexey Ivanov aivanov at openjdk.org
Mon Feb 24 17:04:54 UTC 2025


On Thu, 20 Feb 2025 21:55:43 GMT, Phil Race <prr at openjdk.org> wrote:

> > And two @throws with the same type aren't allowed in javadoc, which means you have to add the new reason to throw IllegalArgumentException into the existing one.
> 
> That's demonstrably untrue. This method already has two.

I was wrong here, the specification for [`ICC_Profile.html.setData`](https://docs.oracle.com/en/java/javase/23/docs/api/java.desktop/java/awt/color/ICC_Profile.html#setData(int,byte[])) correctly displays the two entries for `IllegalArgumentException`. The generated javadoc for the proposed changeset contains three entries for IAE.

I misremembered it, or it was a limitation in an IDE javadoc parser.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23606#discussion_r1968059734


More information about the client-libs-dev mailing list