You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by gg...@apache.org on 2013/11/28 04:58:52 UTC

svn commit: r1546277 - /commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/impl/DefaultFileReplicator.java

Author: ggregory
Date: Thu Nov 28 03:58:52 2013
New Revision: 1546277

URL: http://svn.apache.org/r1546277
Log:
[PMD] Random object created and used only once in org.apache.commons.vfs2.impl.DefaultFileReplicator.init().

Modified:
    commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/impl/DefaultFileReplicator.java

Modified: commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/impl/DefaultFileReplicator.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/impl/DefaultFileReplicator.java?rev=1546277&r1=1546276&r2=1546277&view=diff
==============================================================================
--- commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/impl/DefaultFileReplicator.java (original)
+++ commons/proper/vfs/trunk/core/src/main/java/org/apache/commons/vfs2/impl/DefaultFileReplicator.java Thu Nov 28 03:58:52 2013
@@ -41,6 +41,8 @@ public class DefaultFileReplicator
 {
     private static final Log log = LogFactory.getLog(DefaultFileReplicator.class);
     private static final int MASK = 0xffff;
+    
+    private static final Random random = new Random();
 
     private static final char[] TMP_RESERVED_CHARS = new char[]
         {
@@ -192,7 +194,7 @@ public class DefaultFileReplicator
             tempDir = new File(baseTmpDir, "vfs_cache").getAbsoluteFile();
         }
 
-        filecount = new Random().nextInt() & MASK;
+        filecount = random.nextInt() & MASK;
 
         if (!tempDirMessageLogged)
         {