Code Review 7104209: Cleanup and remove librmi (native library)
Chris Hegarty
chris.hegarty at oracle.com
Tue Oct 25 08:19:38 UTC 2011
D'oh! Sorry my fault.
The changes look good, and thanks for taking care of this.
-Chris.
On 10/25/11 09:08 AM, 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