On Thu, 7 Aug 2025 16:06:35 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Nizar Benalla has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Feedback from Alexey.
>
> src/java.base/share/classes/java/lang/UnsupportedOperationException.java line
> 81:
>
>> 79: * Constructs a new {@code UnsupportedOperationException} with the
>> specified cause and a detail
>> 80: * message of {@code (cause==null ? null : cause.toString())} (which
>> 81: * typically contains the class and detail message of {@code cause}).
>
> This one is trickier. Wrapping the line will cause a ripple effect on the
> following lines, which I'd like to avoid or to minimise at least.
> Suggestion:
>
> * Constructs a new {@code UnsupportedOperationException} with the
> specified
> * cause and a detail message of
> * {@code (cause==null ? null : cause.toString())} (which
> * typically contains the class and detail message of {@code cause}).
FYI I will be clarifying the javadoc of the other subclasses of
RuntimeException as well
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26533#discussion_r2261163670
