On Thu, 16 Jan 2025 01:04:39 GMT, Alex Menkov <amen...@openjdk.org> wrote:

>> Kevin Walls has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   test update/simplification
>
> test/jdk/javax/management/mxbean/SameObjectTwoNamesTest.java line 43:
> 
>> 41: 
>> 42:     public static void main(String[] args) throws Exception {
>> 43:         boolean expectException = true;
> 
> I think it would be better to drop this variable at all (and remove 
> unreachable code when the variable is "false")

Yes sure, can remove that and simplify this small test - done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23131#discussion_r1918113685

Reply via email to