RFR: 8307244: Remove redundant class RMIIIOPServerImpl

Daniel Fuchs dfuchs at openjdk.org
Wed May 3 11:31:14 UTC 2023


On Tue, 2 May 2023 17:57:14 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.

Marked as reviewed by dfuchs (Reviewer).

Looks good to me. I probably wouldn't have bothered with removing the example of IIOP JMXServiceURL as arguably functional implementations of that might still exist (as long as they don't extend RMIIIOPServerImpl, which is not a requirement).
The constructor of  RMIIIOPServerImpl throws UnsupportedOperationException unconditionally, so there can't exist any functional subclasses of that class that could be instantiated on Java versions posterior to Java 9.
As such removing that class from the public API sounds reasonable.

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

PR Review: https://git.openjdk.org/jdk/pull/13758#pullrequestreview-1410697538
PR Comment: https://git.openjdk.org/jdk/pull/13758#issuecomment-1532865857


More information about the serviceability-dev mailing list