RFR(xs): 8066633: Fix deprecation warnings in java.rmi module

joe darcy joe.darcy at oracle.com
Fri Dec 12 01:57:09 UTC 2014


Hi Stuart,

Looks good!

Thanks for fixing this,

-Joe

On 12/11/2014 5:56 PM, Stuart Marks wrote:
> Hi all,
>
> Please review this tiny fix to remove the source of a deprecation 
> warning and to remove the now-unnecessary suppression of deprecation 
> warnings from its context.
>
> There are other deprecated usages mentioned in this bug report but for 
> a variety of reasons they either cannot be removed or aren't worth 
> removing. This instance is the only one left.
>
> Patch appended below.
>
> Bug:
>     https://bugs.openjdk.java.net/browse/JDK-8066633
>
> Thanks,
>
> s'marks
>
>
> # HG changeset patch
> # User smarks
> # Date 1418349197 28800
> #      Thu Dec 11 17:53:17 2014 -0800
> # Node ID 04cfafe160978b6f99f9609f0ad559d4f70829af
> # Parent  420a1b6caaf4114e082c55faf63c67b20337528c
> 8066633: Fix deprecation warnings in java.rmi module
> Reviewed-by: XXX
>
> diff -r 420a1b6caaf4 -r 04cfafe16097 
> src/java.rmi/share/classes/sun/rmi/registry/RegistryImpl.java
> --- a/src/java.rmi/share/classes/sun/rmi/registry/RegistryImpl.java 
> Thu Dec 11 17:09:46 2014 -0800
> +++ b/src/java.rmi/share/classes/sun/rmi/registry/RegistryImpl.java 
> Thu Dec 11 17:53:17 2014 -0800
> @@ -336,13 +336,12 @@
>       * Main program to start a registry. <br>
>       * The port number can be specified on the command line.
>       */
> -    @SuppressWarnings("deprecation")
>      public static void main(String args[])
>      {
>          // Create and install the security manager if one is not 
> installed
>          // already.
>          if (System.getSecurityManager() == null) {
> -            System.setSecurityManager(new RMISecurityManager());
> +            System.setSecurityManager(new SecurityManager());
>          }
>
>          try {




More information about the core-libs-dev mailing list