+1 On Feb 12, 2014, at 8:19 AM, Alan Bateman <alan.bate...@oracle.com> wrote:
> > I need a reviewer for a trivial change to remove a tiny number of unused > imports. The motive is experimental compilation of the JDK as modules rather > than one big compilation unit. I've no doubt that there is other code that > has unused imports but it's not the goal of this exercise to identify and > remove these. The proposal patch is below. > > -Alan > > > diff --git a/src/share/classes/java/lang/invoke/MethodHandle.java > b/src/share/classes/java/lang/invoke/MethodHandle.java > --- a/src/share/classes/java/lang/invoke/MethodHandle.java > +++ b/src/share/classes/java/lang/invoke/MethodHandle.java > @@ -31,8 +31,6 @@ > import sun.misc.Unsafe; > > import static java.lang.invoke.MethodHandleStatics.*; > -import java.util.logging.Level; > -import java.util.logging.Logger; > > /** > * A method handle is a typed, directly executable reference to an underlying > method, > diff --git a/src/share/classes/java/lang/invoke/SimpleMethodHandle.java > b/src/share/classes/java/lang/invoke/SimpleMethodHandle.java > --- a/src/share/classes/java/lang/invoke/SimpleMethodHandle.java > +++ b/src/share/classes/java/lang/invoke/SimpleMethodHandle.java > @@ -27,8 +27,6 @@ > > import static java.lang.invoke.LambdaForm.*; > import static java.lang.invoke.MethodHandleNatives.Constants.*; > -import java.util.logging.Level; > -import java.util.logging.Logger; > > /** > * A method handle whose behavior is determined only by its LambdaForm. > diff --git a/src/share/classes/sun/applet/AppletViewerPanel.java > b/src/share/classes/sun/applet/AppletViewerPanel.java > --- a/src/share/classes/sun/applet/AppletViewerPanel.java > +++ b/src/share/classes/sun/applet/AppletViewerPanel.java > @@ -31,7 +31,6 @@ > import java.net.MalformedURLException; > import java.awt.*; > import java.applet.*; > -import sun.tools.jar.*; Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com