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