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"));
> }
>
> /**
>
