Changeset 1403 for branches/eraser6/BlackBox

Timestamp:

12/22/2009 3:47:42 AM (3 years ago)

Author:

lowjoel

Message:

Decide once the length of the boundary since placing the Random call in the loop condition would introduce too much unpredictability in the boundary length.

File:

• branches/eraser6/BlackBox/Eraser/BlackBoxUploadForm.cs (modified) (2 diffs)

branches/eraser6/BlackBox/Eraser/BlackBoxUploadForm.cs

@@ -248 +248 @@
             {
                 Random rand = new Random();
-                for (int i = 0; i < 20 + rand.Next(40); ++i)
+                for (int i = 0, j = 20 + rand.Next(40); i < j; ++i)
                     Boundary += ValidBoundaryChars[rand.Next(ValidBoundaryChars.Length)];
             }
@@ -312 +312 @@
         private string FileName;
 
+        /// <summary>
+        /// Characters valid for use in the multipart boundary.
+        /// </summary>
         private static readonly string ValidBoundaryChars =
             "0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ'()+_,-./:=?";