Re: RFR: 8026427: deprecate obsolete APIs from java.rmi

2013-10-23 Thread Darryl Mocek
With the previous comments in mind, looks OK to me. Darryl On 10/21/2013 07:07 PM, 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: http

Re: RFR: 8026427: deprecate obsolete APIs from java.rmi

2013-10-22 Thread Stuart Marks
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.

Re: RFR: 8026427: deprecate obsolete APIs from java.rmi

2013-10-22 Thread Stuart Marks
On 10/22/13 5:18 AM, Alan Bateman wrote: On 22/10/2013 03:07, 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

Re: RFR: 8026427: deprecate obsolete APIs from java.rmi

2013-10-22 Thread Daniel Fuchs
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.getSecurityManage

Re: RFR: 8026427: deprecate obsolete APIs from java.rmi

2013-10-22 Thread Alan Bateman
On 22/10/2013 03:07, 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:

RFR: 8026427: deprecate obsolete APIs from java.rmi

2013-10-21 Thread Stuart Marks
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/80264