On Wed, 10 May 2023 08:49:47 GMT, Kevin Walls <kev...@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

Reply via email to