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 2014/04/03 03:20:31 UTC

svn commit: r1584244 - /commons/proper/jcs/trunk/src/java/org/apache/commons/jcs/auxiliary/disk/block/BlockDisk.java

Author: sebb
Date: Thu Apr  3 01:20:31 2014
New Revision: 1584244

URL: http://svn.apache.org/r1584244
Log:
Private immutable fields might as well be final

Modified:
    commons/proper/jcs/trunk/src/java/org/apache/commons/jcs/auxiliary/disk/block/BlockDisk.java

Modified: commons/proper/jcs/trunk/src/java/org/apache/commons/jcs/auxiliary/disk/block/BlockDisk.java
URL: http://svn.apache.org/viewvc/commons/proper/jcs/trunk/src/java/org/apache/commons/jcs/auxiliary/disk/block/BlockDisk.java?rev=1584244&r1=1584243&r2=1584244&view=diff
==============================================================================
--- commons/proper/jcs/trunk/src/java/org/apache/commons/jcs/auxiliary/disk/block/BlockDisk.java (original)
+++ commons/proper/jcs/trunk/src/java/org/apache/commons/jcs/auxiliary/disk/block/BlockDisk.java Thu Apr  3 01:20:31 2014
@@ -55,7 +55,7 @@ public class BlockDisk
     private static final int DEFAULT_BLOCK_SIZE_BYTES = 4 * 1024;
 
     /** Size of the blocks */
-    private int blockSizeBytes;
+    private final int blockSizeBytes;
 
     /**
      * the total number of blocks that have been used. If there are no free, we will use this to
@@ -67,7 +67,7 @@ public class BlockDisk
     private final SingleLinkedList<Integer> emptyBlocks = new SingleLinkedList<Integer>();
 
     /** The serializer. */
-    protected IElementSerializer elementSerializer;
+    private final IElementSerializer elementSerializer;
 
     /** Location of the spot on disk */
     private final String filepath;