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/02/15 19:12:50 UTC

svn commit: r1568655 - in /commons/proper/fileupload/trunk/src: changes/changes.xml main/java/org/apache/commons/fileupload/MultipartStream.java

Author: sebb
Date: Sat Feb 15 18:12:49 2014
New Revision: 1568655

URL: http://svn.apache.org/r1568655
Log:
FILEUPLOAD-243 Make some MultipartStream private fields final

Modified:
    commons/proper/fileupload/trunk/src/changes/changes.xml
    commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java

Modified: commons/proper/fileupload/trunk/src/changes/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/fileupload/trunk/src/changes/changes.xml?rev=1568655&r1=1568654&r2=1568655&view=diff
==============================================================================
--- commons/proper/fileupload/trunk/src/changes/changes.xml (original)
+++ commons/proper/fileupload/trunk/src/changes/changes.xml Sat Feb 15 18:12:49 2014
@@ -43,6 +43,11 @@ The <action> type attribute can be add,u
   </properties>
 
   <body>
+    <release version="1.4" date="TBA" description=
+"TBA">
+      <action dev="sebb" type="update" due-to="Ville Skyttä">Make some MultipartStream private fields final</action>
+    </release>
+
     <release version="1.3.1" description=
 "This is a security and maintenance release that includes an important security 
 fix as well as a small number of bugfixes." date="TBD">

Modified: commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java
URL: http://svn.apache.org/viewvc/commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java?rev=1568655&r1=1568654&r2=1568655&view=diff
==============================================================================
--- commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java (original)
+++ commons/proper/fileupload/trunk/src/main/java/org/apache/commons/fileupload/MultipartStream.java Sat Feb 15 18:12:49 2014
@@ -222,12 +222,12 @@ public class MultipartStream {
      * The amount of data, in bytes, that must be kept in the buffer in order
      * to detect delimiters reliably.
      */
-    private int keepRegion;
+    private final int keepRegion;
 
     /**
      * The byte sequence that partitions the stream.
      */
-    private byte[] boundary;
+    private final byte[] boundary;
 
     /**
      * The length of the buffer used for processing the request.