Some automatic code cleanups

Author: slachiewicz

src/main/java/org/codehaus/classworlds/BytesURLConnection.java

@@ -25,7 +25,7 @@
  */
 @Deprecated
 public class BytesURLConnection extends URLConnection {
-    protected byte[] content;
+    protected final byte[] content;
 
     protected int offset;
 

src/main/java/org/codehaus/classworlds/BytesURLStreamHandler.java

@@ -27,7 +27,7 @@
  */
 @Deprecated
 public class BytesURLStreamHandler extends URLStreamHandler {
-    byte[] content;
+    final byte[] content;
 
     int offset;
 

src/main/java/org/codehaus/classworlds/ClassRealm.java

@@ -60,6 +60,7 @@ STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * @author <a href="mailto:[email protected]">bob mcwhirter</a>
  * @author <a href="mailto:[email protected]">Jason van Zyl</a>
  */
+@SuppressWarnings("rawtypes")
 @Deprecated
 public interface ClassRealm {
     String getId();

src/main/java/org/codehaus/classworlds/ClassRealmAdapter.java

@@ -26,6 +26,7 @@
  *
  * @author Andrew Williams
  */
+@SuppressWarnings({"UnnecessaryLocalVariable", "DeprecatedIsStillUsed", "rawtypes"})
 @Deprecated
 public class ClassRealmAdapter implements ClassRealm {
 
@@ -35,7 +36,7 @@ public static ClassRealmAdapter getInstance(org.codehaus.plexus.classworlds.real
         return adapter;
     }
 
-    private org.codehaus.plexus.classworlds.realm.ClassRealm realm;
+    private final org.codehaus.plexus.classworlds.realm.ClassRealm realm;
 
     private ClassRealmAdapter(org.codehaus.plexus.classworlds.realm.ClassRealm newRealm) {
         this.realm = newRealm;

src/main/java/org/codehaus/classworlds/ClassRealmReverseAdapter.java

@@ -26,6 +26,7 @@
  *
  * @author Andrew Williams
  */
+@SuppressWarnings({"UnnecessaryLocalVariable", "rawtypes"})
 @Deprecated
 public class ClassRealmReverseAdapter extends org.codehaus.plexus.classworlds.realm.ClassRealm {
 
@@ -35,7 +36,7 @@ public static ClassRealmReverseAdapter getInstance(ClassRealm oldRealm) {
         return adapter;
     }
 
-    private ClassRealm realm;
+    private final ClassRealm realm;
 
     private ClassRealmReverseAdapter(ClassRealm oldRealm) {
         super(ClassWorldReverseAdapter.getInstance(oldRealm.getWorld()), oldRealm.getId(), oldRealm.getClassLoader());
@@ -100,6 +101,7 @@ public URL getResource(String name) {
         return realm.getResource(name);
     }
 
+    @SuppressWarnings("unchecked")
     public Enumeration findResources(String name) throws IOException {
         return realm.findResources(name);
     }

src/main/java/org/codehaus/classworlds/ClassWorld.java

@@ -24,6 +24,7 @@
  *
  * @author Andrew Williams
  */
+@SuppressWarnings("rawtypes")
 @Deprecated
 public class ClassWorld {
     private ClassWorldAdapter adapter;

src/main/java/org/codehaus/classworlds/ClassWorldAdapter.java

@@ -24,14 +24,15 @@
  *
  * @author Andrew Williams
  */
+@SuppressWarnings("rawtypes")
 @Deprecated
 public class ClassWorldAdapter extends ClassWorld {
 
     public static ClassWorldAdapter getInstance(org.codehaus.plexus.classworlds.ClassWorld newWorld) {
         return new ClassWorldAdapter(newWorld);
     }
 
-    private org.codehaus.plexus.classworlds.ClassWorld world;
+    private final org.codehaus.plexus.classworlds.ClassWorld world;
 
     private ClassWorldAdapter(org.codehaus.plexus.classworlds.ClassWorld newWorld) {
         super(false);

src/main/java/org/codehaus/classworlds/ClassWorldException.java

@@ -59,7 +59,7 @@ public class ClassWorldException extends Exception {
     /**
      * The world.
      */
-    private ClassWorld world;
+    private final ClassWorld world;
 
     // ------------------------------------------------------------
     //     Constructors

src/main/java/org/codehaus/classworlds/ClassWorldReverseAdapter.java

@@ -25,9 +25,10 @@
  *
  * @author Andrew Williams
  */
+@SuppressWarnings({"unchecked", "rawtypes"})
 @Deprecated
 public class ClassWorldReverseAdapter extends org.codehaus.plexus.classworlds.ClassWorld {
-    private static HashMap instances = new HashMap();
+    private static final HashMap instances = new HashMap();
 
     public static ClassWorldReverseAdapter getInstance(ClassWorld oldWorld) {
         if (instances.containsKey(oldWorld)) return (ClassWorldReverseAdapter) instances.get(oldWorld);
@@ -38,7 +39,7 @@ public static ClassWorldReverseAdapter getInstance(ClassWorld oldWorld) {
         return adapter;
     }
 
-    private ClassWorld world;
+    private final ClassWorld world;
 
     private ClassWorldReverseAdapter(ClassWorld newWorld) {
         super();

src/main/java/org/codehaus/classworlds/Configurator.java

@@ -29,7 +29,7 @@
  */
 @Deprecated
 public class Configurator {
-    private ConfiguratorAdapter config;
+    private final ConfiguratorAdapter config;
 
     /** Construct.
      *
@@ -108,6 +108,7 @@ protected void loadGlob(String line, ClassRealm realm) throws MalformedURLExcept
      * @throws FileNotFoundException If the line does not represent
      *                               a valid path element in the filesystem.
      */
+    @SuppressWarnings("RedundantThrows")
     protected void loadGlob(String line, ClassRealm realm, boolean optionally)
             throws MalformedURLException, FileNotFoundException {
         config.loadGlob(line, realm, optionally);
@@ -121,6 +122,7 @@ protected void loadGlob(String line, ClassRealm realm, boolean optionally)
      * @throws ConfigurationException If the property does not
      *                                exist or if there is a syntax error.
      */
+    @SuppressWarnings("RedundantThrows")
     protected String filter(String text) throws ConfigurationException {
         return config.filter(text);
     }