RFR: 8359809: AttributeList, RoleList and UnresolvedRoleList should never accept other types of Object [v5]
Kevin Walls
kevinw at openjdk.org
Thu Jul 10 08:11:41 UTC 2025
On Wed, 9 Jul 2025 23:16:12 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> Kevin Walls has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Further doc update
>> - Test udpate (listIterator)
>
> test/jdk/javax/management/generified/ListTypeCheckTest.java line 110:
>
>> 108: throw new Exception("test wrong");
>> 109: }
>> 110: throw new Exception("op " + i + " allowed but should fail on " + al.getClass());
>
> Nit: This is confusing. If it is allowed then why should it fail.
> At least, it needs to be explained somewhere in the tests.
Thanks Serguei, I hope I've addressed both comments with additional comment updates.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25856#discussion_r2196935071
More information about the serviceability-dev
mailing list