RFR: 8026427: deprecate obsolete APIs from java.rmi

Stuart Marks stuart.marks at oracle.com
Tue Oct 22 20:18:14 UTC 2013


Hi Daniel,

Yes, this round is just deprecation, not removal, so anything that uses 
RMISecurityManager can continue to do so for the near future.

Ideally we should remove all references to RMISecurityManager from the JDK, but 
this can be done later. The only cost is some deprecation warnings. (Such 
warnings are disabled at the moment.)

RMISecurityManager is an empty subclass of SecurityManager, thus there are no 
semantic differences. It's been this way since around 1.3. There are probably 
applications out there that construct RMISecurityManager. At least, some RMI 
tutorials advise doing this (implying that it's necessary for some reason). 
Deprecation should flush out these usages.

I suppose somebody could have written an application that downcasts to 
RMISecurityManager, or does instanceof RMISecurityManager, but there would be no 
point in doing so, since there are no APIs revealed by doing so. Thus I think it 
would be fairly safe to change all usages of RMISecurityManager to 
SecurityManager in the JDK.

s'marks

On 10/22/13 5:58 AM, Daniel Fuchs wrote:
> Hi Stuart,
>
> This looks good to me.
>
> RMISecurityManager:
>
> I see that there are a few places in the JDK where we still
> instantiate an RMISecurityManager.
>
> I assume we must unfortunately keep them for backward compatibility,
> in case some application code would try to downcast
> System.getSecurityManager() to RMISecurityManager?
>
> best regards,
>
> -- daniel (not a reviewer)
>
> On 10/22/13 4:07 AM, Stuart Marks wrote:
>> Hi all,
>>
>> Please review this small spec change to deprecate some obsolete RMI
>> APIs, along with a bit of associated cleanup. There are no functional
>> changes in this changeset.
>>
>> Bug:
>>
>>      https://bugs.openjdk.java.net/browse/JDK-8026427
>>
>> Webrev:
>>
>>      http://cr.openjdk.java.net/~smarks/reviews/8026427/webrev.0/
>>
>> Specdiff:
>>
>>
>> http://cr.openjdk.java.net/~smarks/reviews/8026427/specdiff.0/overview-summary.html
>>
>>
>>
>> Thanks,
>>
>> s'marks
>



More information about the core-libs-dev mailing list