On Tue, 11 Jun 2024 16:55:44 GMT, Weijun Wang <[email protected]> wrote:
>> Kevin Walls has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Sean comments
>
> src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java
> line 1633:
>
>> 1631: }
>> 1632: } else {
>> 1633: // ACC is present, we have a Subject and SM is
>> permitted:
>
> While extract the `action` variable? The old code on lines 1590-1592 has no
> problem.
OK I can make this more like the original.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19624#discussion_r1636515151
