7151348: Build breaks due to warning clean up in sun.rmi.*(7146763)
authorkhazra
Mon, 05 Mar 2012 17:38:01 -0800
changeset 12045 3c08f7ce2890
parent 12044 d826961f7d42
child 12046 378aa3362868
7151348: Build breaks due to warning clean up in sun.rmi.*(7146763) Summary: Undo changes to sun/rmi/rmic/* commited as fix for 7146763 Reviewed-by: smarks
jdk/src/share/classes/sun/rmi/rmic/BatchEnvironment.java
jdk/src/share/classes/sun/rmi/rmic/Main.java
jdk/src/share/classes/sun/rmi/rmic/RMIGenerator.java
jdk/src/share/classes/sun/rmi/rmic/newrmic/Main.java
jdk/src/share/classes/sun/rmi/rmic/newrmic/Resources.java
--- a/jdk/src/share/classes/sun/rmi/rmic/BatchEnvironment.java	Mon Mar 05 09:33:44 2012 -0800
+++ b/jdk/src/share/classes/sun/rmi/rmic/BatchEnvironment.java	Mon Mar 05 17:38:01 2012 -0800
@@ -160,7 +160,7 @@
     }
 
     /** list of generated source files created in this environment */
-    private Vector<File> generatedFiles = new Vector<>();
+    private Vector generatedFiles = new Vector();
 
     /**
      * Remember a generated source file generated so that it
@@ -177,9 +177,9 @@
      */
     public void deleteGeneratedFiles() {
         synchronized(generatedFiles) {
-            Enumeration<File> enumeration = generatedFiles.elements();
+            Enumeration enumeration = generatedFiles.elements();
             while (enumeration.hasMoreElements()) {
-                File file = enumeration.nextElement();
+                File file = (File) enumeration.nextElement();
                 file.delete();
             }
             generatedFiles.removeAllElements();
--- a/jdk/src/share/classes/sun/rmi/rmic/Main.java	Mon Mar 05 09:33:44 2012 -0800
+++ b/jdk/src/share/classes/sun/rmi/rmic/Main.java	Mon Mar 05 17:38:01 2012 -0800
@@ -73,15 +73,14 @@
     File destDir;
     int flags;
     long tm;
-    Vector<String> classes;
+    Vector classes;
     boolean nowrite;
     boolean nocompile;
     boolean keepGenerated;
     boolean status;
     String[] generatorArgs;
-    Vector<Generator> generators;
-    Class<? extends BatchEnvironment> environmentClass =
-        BatchEnvironment.class;
+    Vector generators;
+    Class environmentClass = BatchEnvironment.class;
     boolean iiopGeneration = false;
 
     /**
@@ -184,7 +183,7 @@
         destDir = null;
         flags = F_WARNINGS;
         tm = System.currentTimeMillis();
-        classes = new Vector<>();
+        classes = new Vector();
         nowrite = false;
         nocompile = false;
         keepGenerated = false;
@@ -192,7 +191,7 @@
         if (generatorArgs == null) {
             return false;
         }
-        generators = new Vector<>();
+        generators = new Vector();
 
         // Pre-process command line for @file arguments
         try {
@@ -412,7 +411,7 @@
 
         // Get the environment required by this generator...
 
-        Class<?> envClass = BatchEnvironment.class;
+        Class envClass = BatchEnvironment.class;
         String env = getString("generator.env." + arg);
         if (env != null) {
             try {
@@ -424,7 +423,7 @@
 
                     // Yes, so switch to the new one...
 
-                    environmentClass = BatchEnvironment.class.asSubclass(environmentClass);
+                    environmentClass = envClass;
 
                 } else {
 
@@ -496,9 +495,8 @@
         try {
             Class[] ctorArgTypes = {OutputStream.class,ClassPath.class,Main.class};
             Object[] ctorArgs = {out,classPath,this};
-            Constructor<? extends BatchEnvironment> constructor =
-                environmentClass.getConstructor(ctorArgTypes);
-            result =  constructor.newInstance(ctorArgs);
+            Constructor constructor = environmentClass.getConstructor(ctorArgTypes);
+            result = (BatchEnvironment) constructor.newInstance(ctorArgs);
             result.reset();
         }
         catch (Exception e) {
@@ -532,7 +530,7 @@
              */
             for (int i = classes.size()-1; i >= 0; i-- ) {
                 Identifier implClassName =
-                    Identifier.lookup(classes.elementAt(i));
+                    Identifier.lookup((String)classes.elementAt(i));
 
                 /*
                  * Fix bugid 4049354: support using '.' as an inner class
@@ -560,7 +558,7 @@
                 try {
                     ClassDefinition def = decl.getClassDefinition(env);
                     for (int j = 0; j < generators.size(); j++) {
-                        Generator gen = generators.elementAt(j);
+                        Generator gen = (Generator)generators.elementAt(j);
                         gen.generate(env, def, destDir);
                     }
                 } catch (ClassNotFound ex) {
@@ -675,7 +673,7 @@
 
         do {
             done = true;
-            for (Enumeration<?> e = env.getClasses() ; e.hasMoreElements() ; ) {
+            for (Enumeration e = env.getClasses() ; e.hasMoreElements() ; ) {
                 ClassDeclaration c = (ClassDeclaration)e.nextElement();
                 done = compileClass(c,buf,env);
             }
@@ -684,9 +682,7 @@
 
     /*
      * Compile a single class.
-     * Fallthrough is intentional
      */
-    @SuppressWarnings("fallthrough")
     public boolean compileClass (ClassDeclaration c,
                                  ByteArrayOutputStream buf,
                                  BatchEnvironment env)
@@ -883,6 +879,6 @@
         args[1] = (arg1 != null ? arg1.toString() : "null");
         args[2] = (arg2 != null ? arg2.toString() : "null");
 
-        return java.text.MessageFormat.format(format, (Object[]) args);
+        return java.text.MessageFormat.format(format, args);
     }
 }
--- a/jdk/src/share/classes/sun/rmi/rmic/RMIGenerator.java	Mon Mar 05 09:33:44 2012 -0800
+++ b/jdk/src/share/classes/sun/rmi/rmic/RMIGenerator.java	Mon Mar 05 17:38:01 2012 -0800
@@ -61,7 +61,7 @@
  */
 public class RMIGenerator implements RMIConstants, Generator {
 
-    private static final Hashtable<String, Integer> versionOptions = new Hashtable<>();
+    private static final Hashtable versionOptions = new Hashtable();
     static {
         versionOptions.put("-v1.1", new Integer(STUB_VERSION_1_1));
         versionOptions.put("-vcompat", new Integer(STUB_VERSION_FAT));
@@ -96,7 +96,7 @@
                         return false;
                     }
                     explicitVersion = arg;
-                    version = versionOptions.get(arg);
+                    version = ((Integer) versionOptions.get(arg)).intValue();
                     argv[i] = null;
                 }
             }
@@ -519,7 +519,7 @@
          * follows a previous catch of it or of one of its superclasses.
          * The following method invocation takes care of these details.
          */
-        Vector<ClassDefinition> catchList = computeUniqueCatchList(exceptions);
+        Vector catchList = computeUniqueCatchList(exceptions);
 
         /*
          * If we need to catch any particular exceptions (i.e. this method
@@ -615,10 +615,10 @@
          * UnexpectedException, and end the try block.
          */
         if (catchList.size() > 0) {
-            for (Enumeration<ClassDefinition> enumeration = catchList.elements();
+            for (Enumeration enumeration = catchList.elements();
                  enumeration.hasMoreElements();)
             {
-                ClassDefinition def = enumeration.nextElement();
+                ClassDefinition def = (ClassDefinition) enumeration.nextElement();
                 p.pOlnI("} catch (" + def.getName() + " e) {");
                 p.pln("throw e;");
             }
@@ -650,8 +650,8 @@
      * of its superclasses is in the throws clause of the method, indicating
      * that no exceptions need to be caught.
      */
-    private Vector<ClassDefinition> computeUniqueCatchList(ClassDeclaration[] exceptions) {
-        Vector<ClassDefinition> uniqueList = new Vector<>();       // unique exceptions to catch
+    private Vector computeUniqueCatchList(ClassDeclaration[] exceptions) {
+        Vector uniqueList = new Vector();       // unique exceptions to catch
 
         uniqueList.addElement(defRuntimeException);
         uniqueList.addElement(defRemoteException);
@@ -682,7 +682,8 @@
                  * exceptions that need to be caught:
                  */
                 for (int j = 0; j < uniqueList.size();) {
-                    ClassDefinition def = uniqueList.elementAt(j);
+                    ClassDefinition def =
+                        (ClassDefinition) uniqueList.elementAt(j);
                     if (def.superClassOf(env, decl)) {
                         /*
                          * If a superclass of this exception is already on
--- a/jdk/src/share/classes/sun/rmi/rmic/newrmic/Main.java	Mon Mar 05 09:33:44 2012 -0800
+++ b/jdk/src/share/classes/sun/rmi/rmic/newrmic/Main.java	Mon Mar 05 17:38:01 2012 -0800
@@ -455,7 +455,7 @@
         BatchEnvironment env;
         try {
             Constructor<? extends BatchEnvironment> cons =
-                batch.envClass.getConstructor(new Class<?>[] { RootDoc.class });
+                batch.envClass.getConstructor(new Class[] { RootDoc.class });
             env = cons.newInstance(rootDoc);
         } catch (NoSuchMethodException e) {
             throw new AssertionError(e);
--- a/jdk/src/share/classes/sun/rmi/rmic/newrmic/Resources.java	Mon Mar 05 09:33:44 2012 -0800
+++ b/jdk/src/share/classes/sun/rmi/rmic/newrmic/Resources.java	Mon Mar 05 17:38:01 2012 -0800
@@ -69,7 +69,7 @@
             format = "missing resource key: key = \"" + key + "\", " +
                 "arguments = \"{0}\", \"{1}\", \"{2}\"";
         }
-        return MessageFormat.format(format, (Object[]) args);
+        return MessageFormat.format(format, args);
     }
 
     /**