svn commit: r1586919 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/utils/IOUtils.java

[bodewig]

Author: bodewig
Date: Sun Apr 13 04:07:20 2014
New Revision: 1586919

URL: http://svn.apache.org/r1586919
Log:
COMPRESS-277 share the buffer used in skip - concurrent writes to the buffer 
won't do any harm as the data is discarded ayway

Modified:
    
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/utils/IOUtils.java

Modified: 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/utils/IOUtils.java
URL: 
http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/utils/IOUtils.java?rev=1586919&r1=1586918&r2=1586919&view=diff
==============================================================================
--- 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/utils/IOUtils.java
 (original)
+++ 
commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/utils/IOUtils.java
 Sun Apr 13 04:07:20 2014
@@ -32,6 +32,7 @@ public final class IOUtils {
 
     private static final int COPY_BUF_SIZE = 8024;
     private static final int SKIP_BUF_SIZE = 4096;
+    private static final byte[] SKIP_BUF = new byte[SKIP_BUF_SIZE];
 
     /** Private constructor to prevent instantiation of this utility class. */
     private IOUtils(){
@@ -101,16 +102,13 @@ public final class IOUtils {
             numToSkip -= skipped;
         }
             
-        if (numToSkip > 0) {
-            byte[] skipBuf = new byte[SKIP_BUF_SIZE];
-            while (numToSkip > 0) {
-                int read = readFully(input, skipBuf, 0,
-                                     (int) Math.min(numToSkip, SKIP_BUF_SIZE));
-                if (read < 1) {
-                    break;
-                }
-                numToSkip -= read;
+        while (numToSkip > 0) {
+            int read = readFully(input, SKIP_BUF, 0,
+                                 (int) Math.min(numToSkip, SKIP_BUF_SIZE));
+            if (read < 1) {
+                break;
             }
+            numToSkip -= read;
         }
         return available - numToSkip;
     }