Author: sebb
Date: Tue Apr  5 21:28:03 2011
New Revision: 1089236

URL: http://svn.apache.org/viewvc?rev=1089236&view=rev
Log:
Private field might as well be final

Modified:
    
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/SPInterface.java

Modified: 
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/SPInterface.java
URL: 
http://svn.apache.org/viewvc/commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/SPInterface.java?rev=1089236&r1=1089235&r2=1089236&view=diff
==============================================================================
--- 
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/SPInterface.java
 (original)
+++ 
commons/proper/discovery/trunk/src/java/org/apache/commons/discovery/tools/SPInterface.java
 Tue Apr  5 21:28:03 2011
@@ -46,9 +46,9 @@ public class SPInterface<T> {
      */
     private final String propertyName;
 
-    private Class<?>  paramClasses[] = null;
+    private final Class<?>  paramClasses[];
 
-    private Object params[] = null;
+    private final Object params[];
 
     /**
      * Construct object representing Class <code>provider</code>.
@@ -72,6 +72,8 @@ public class SPInterface<T> {
     public SPInterface(Class<T> spi, String propertyName) {
         this.spi = spi;
         this.propertyName = propertyName;
+        this.paramClasses = null;
+        this.params = null;
     }
 
     /**


Reply via email to