You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2009/03/14 17:04:17 UTC

svn commit: r754471 - in /commons/proper/lang/trunk/src: java/org/apache/commons/lang/builder/HashCodeBuilder.java java/org/apache/commons/lang/builder/ToStringStyle.java test/org/apache/commons/lang/BitFieldTest.java

Author: sebb
Date: Sat Mar 14 16:04:17 2009
New Revision: 754471

URL: http://svn.apache.org/viewvc?rev=754471&view=rev
Log:
Make private fields final

Modified:
    commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java
    commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/ToStringStyle.java
    commons/proper/lang/trunk/src/test/org/apache/commons/lang/BitFieldTest.java

Modified: commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java?rev=754471&r1=754470&r2=754471&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java (original)
+++ commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/HashCodeBuilder.java Sat Mar 14 16:04:17 2009
@@ -102,7 +102,7 @@
      * 
      * @since 2.3
      */
-    private static ThreadLocal registry = new ThreadLocal() {
+    private static final ThreadLocal registry = new ThreadLocal() {
         @Override
         protected Object initialValue() {
             // The HashSet implementation is not synchronized,

Modified: commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/ToStringStyle.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/ToStringStyle.java?rev=754471&r1=754470&r2=754471&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/ToStringStyle.java (original)
+++ commons/proper/lang/trunk/src/java/org/apache/commons/lang/builder/ToStringStyle.java Sat Mar 14 16:04:17 2009
@@ -133,7 +133,7 @@
      * to detect cyclical object references and avoid infinite loops.
      * </p>
      */
-    private static ThreadLocal registry = new ThreadLocal() {
+    private static final ThreadLocal registry = new ThreadLocal() {
         @Override
         protected Object initialValue() {
             // The HashSet implementation is not synchronized,

Modified: commons/proper/lang/trunk/src/test/org/apache/commons/lang/BitFieldTest.java
URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/test/org/apache/commons/lang/BitFieldTest.java?rev=754471&r1=754470&r2=754471&view=diff
==============================================================================
--- commons/proper/lang/trunk/src/test/org/apache/commons/lang/BitFieldTest.java (original)
+++ commons/proper/lang/trunk/src/test/org/apache/commons/lang/BitFieldTest.java Sat Mar 14 16:04:17 2009
@@ -41,9 +41,9 @@
         return suite;
     }
 
-    private static BitField bf_multi  = new BitField(0x3F80);
-    private static BitField bf_single = new BitField(0x4000);
-    private static BitField bf_zero = new BitField(0);
+    private static final BitField bf_multi  = new BitField(0x3F80);
+    private static final BitField bf_single = new BitField(0x4000);
+    private static final BitField bf_zero = new BitField(0);
 
     /**
      * Constructor BitFieldTest