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