Code Review 7104209: Cleanup and remove librmi (native library)

Mike Duigou mike.duigou at oracle.com
Tue Oct 25 16:28:23 UTC 2011


Looks good from me as well. Seems like a strange location to have the load library!

Mike

On Oct 25 2011, at 01:08 , Alan Bateman wrote:

> 
> It turns out there's a problem with these changes as it didn't remove the code to load the rmi library (we missed it in the code review too). The result is that most of the RMI tests are now failing. I've just created 7104577 for this. The diffs are trivial  - can I get a reviewer and we'll sweep this under the rug quickly - thanks, Alan.
> 
> diff --git a/src/share/classes/sun/rmi/server/MarshalInputStream.java b/src/share/classes/sun/rmi/server/MarshalInputStream.java
> --- a/src/share/classes/sun/rmi/server/MarshalInputStream.java
> +++ b/src/share/classes/sun/rmi/server/MarshalInputStream.java
> @@ -107,14 +107,6 @@ public class MarshalInputStream extends
>             throw new NoClassDefFoundError("Missing system class: " +
>                                            e.getMessage());
>         }
> -    }
> -
> -    /**
> -     * Load the "rmi" native library.
> -     */
> -    static {
> -        java.security.AccessController.doPrivileged(
> -            new sun.security.action.LoadLibraryAction("rmi"));
>     }
> 
>     /**
> 




More information about the core-libs-dev mailing list