On Mon, 25 Nov 2024 08:08:18 GMT, Alan Bateman <[email protected]> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> minor code comment update in the test
>
> src/java.base/share/classes/sun/net/www/ParseUtil.java line 176:
>
>> 174: * @throws IllegalArgumentException if {@code s} could not be
>> decoded
>> 175: */
>> 176: public static String decode(String s) throws
>> IllegalArgumentException {
>
> I'm not sure if you meant to add this or not but "throws IAE" is not needed,
> usually don't put unchecked exceptions in the throws.
Hello Alan, that was intentional to be more prominently visible, but I'll
remove it and will just retain `@throws` javadoc tag that I added as part of
this change.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22351#discussion_r1856038067
