On Mon, 17 Jun 2024 12:33:08 GMT, Weijun Wang <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - leave noPermissionsACC in place for now
>> - leave noPermissionsACC in place for now
>
> src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
> line 1436:
>
>> 1434: return op.run();
>> 1435: } catch (Exception e) {
>> 1436: if (e instanceof RuntimeException)
>
> Enclose the next line in braces.
Yes this file is full of pre-existing style problems like that. I can update
this and its friend at 1451.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19624#discussion_r1642763028
