Re: Review Request for 8073696: Remove redundant import from javax/rmi/CORBA/GetORBPropertiesFileAction.java

2015-02-24 Thread Chris Hegarty

On 24 Feb 2015, at 00:58, Mandy Chung  wrote:

> In fact, find another redundant in jdk.httpserver.   I have updated the 
> synopsis of JDK-8073696 to remove redundant imports in java.corba and 
> jdk.httpserver
> 
> http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8073696/webrev.00/

Looks good to me.

-Chris.

> Mandy



Re: Review Request for 8073696: Remove redundant import from javax/rmi/CORBA/GetORBPropertiesFileAction.java

2015-02-24 Thread Alan Bateman

On 24/02/2015 00:58, Mandy Chung wrote:
In fact, find another redundant in jdk.httpserver.   I have updated 
the synopsis of JDK-8073696 to remove redundant imports in java.corba 
and jdk.httpserver


http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8073696/webrev.00/

Mandy


Looks okay to me.

-Alan


Re: Review Request for 8073696: Remove redundant import from javax/rmi/CORBA/GetORBPropertiesFileAction.java

2015-02-23 Thread Mandy Chung
In fact, find another redundant in jdk.httpserver.   I have updated the 
synopsis of JDK-8073696 to remove redundant imports in java.corba and 
jdk.httpserver


http://cr.openjdk.java.net/~mchung/jdk9/webrevs/8073696/webrev.00/

Mandy





Review Request for 8073696: Remove redundant import from javax/rmi/CORBA/GetORBPropertiesFileAction.java

2015-02-23 Thread Mandy Chung
diff --git 
a/src/java.corba/share/classes/javax/rmi/CORBA/GetORBPropertiesFileAction.java 
b/src/java.corba/share/classes/javax/rmi/CORBA/GetORBPropertiesFileAction.java
--- 
a/src/java.corba/share/classes/javax/rmi/CORBA/GetORBPropertiesFileAction.java
+++ 
b/src/java.corba/share/classes/javax/rmi/CORBA/GetORBPropertiesFileAction.java

@@ -43,7 +43,6 @@

 import java.security.AccessController;
 import java.security.PrivilegedAction;
-import sun.security.action.GetPropertyAction;
 import java.util.Properties;

 class GetORBPropertiesFileAction implements PrivilegedAction {

Mandy