RFR 8043937: Drop support for the IIOP transport from the JMX RMIConnector

[Jaroslav Bachorik]

On 19.8.2015 15:05, Alan Bateman wrote:
> On 19/08/2015 13:51, Jaroslav Bachorik wrote:
>>
>> I hope I got it all this time. Used brute force search for any mention
>> of 'iiop' and removed anything that had anything to do with JMX.
>>
>> Top level: http://cr.openjdk.java.net/~jbachorik/8043937/webrev.02
>> JDK: http://cr.openjdk.java.net/~jbachorik/8043937/webrev.02/jdk
>
> I think you are close :-)
>
> The @deprecated in class RMIIIOPServerImpl says "This method was ..."
> when it should be "This class". It might be simpler to just drop that
> sentence and say "The IIOP transport is no longer supported".

Ok.

>
> I see the methods in this class throw UOE but I assume that can't happen
> because the constructor always throws an exception (except of course if
> someone does something sneaky, say with finalizers). So I think what you
> have is okay.

Yes, under normal circumstances you should never be able to obtain an 
instance of this class.

>
> There are several tests in jdk/test/javax/management that exercise IIOP
> if available, shouldn't they be updated too?

Not sure about this. These tests are also exercising JMXMP if available 
- but that has never been a part of JDK.

-JB-

>
> -Alan.
>
>