RFR: 8307244: Remove redundant class RMIIIOPServerImpl [v2]

Alan Bateman alanb at openjdk.org
Wed May 10 09:13:42 UTC 2023


On Wed, 10 May 2023 08:49:47 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> Removal of class, looks like it was missed in the JDK9 removal of RMIIIOP.
>> This class is not referenced by other classes or tests.
>
> Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:
> 
>   example update

src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnector.java line 188:

> 186:      *
> 187:      * <pre>
> 188:      * service:jmx:protocolname://<em>[host[:port]]</em>/stub/<em>encoded-stub</em>

"protocolname" could be read as literal here. I think change it back to "iiop" or replace it with something that like "myprotocol" that is more obviously an example protocol name.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13758#discussion_r1189583710


More information about the serviceability-dev mailing list