This is an automated email from the ASF dual-hosted git repository.

peterlee pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-compress.git


The following commit(s) were added to refs/heads/master by this push:
     new 774f6b4  unitized array styles.
     new e587f09  Merge pull request #109 from XenoAmess/array_style
774f6b4 is described below

commit 774f6b41a46fd7ddd2196687115569729646a67a
Author: XenoAmess <xenoam...@gmail.com>
AuthorDate: Fri Jun 5 03:46:38 2020 +0800

    unitized array styles.
---
 .../compress/archivers/cpio/CpioArchiveInputStream.java      |  8 ++++----
 .../compress/archivers/cpio/CpioArchiveOutputStream.java     |  4 ++--
 .../org/apache/commons/compress/archivers/cpio/CpioUtil.java |  2 +-
 .../apache/commons/compress/archivers/sevenz/SevenZFile.java |  2 +-
 .../compress/archivers/zip/X0017_StrongEncryptionHeader.java | 12 ++++++------
 .../org/apache/commons/compress/compressors/bzip2/CRC.java   |  2 +-
 .../apache/commons/compress/archivers/tar/TarUtilsTest.java  |  4 ++--
 .../compress/compressors/bzip2/PythonTruncatedBzip2Test.java |  2 +-
 .../compress/utils/FixedLengthBlockOutputStreamTest.java     |  6 +++---
 9 files changed, 21 insertions(+), 21 deletions(-)

diff --git 
a/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveInputStream.java
 
b/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveInputStream.java
index bec56e5..09d1def 100644
--- 
a/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveInputStream.java
+++ 
b/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveInputStream.java
@@ -75,7 +75,7 @@ public class CpioArchiveInputStream extends 
ArchiveInputStream implements
 
     private boolean entryEOF = false;
 
-    private final byte tmpbuf[] = new byte[4096];
+    private final byte[] tmpbuf = new byte[4096];
 
     private long crc = 0;
 
@@ -355,14 +355,14 @@ public class CpioArchiveInputStream extends 
ArchiveInputStream implements
 
     private long readBinaryLong(final int length, final boolean swapHalfWord)
             throws IOException {
-        final byte tmp[] = new byte[length];
+        final byte[] tmp = new byte[length];
         readFully(tmp, 0, tmp.length);
         return CpioUtil.byteArray2long(tmp, swapHalfWord);
     }
 
     private long readAsciiLong(final int length, final int radix)
             throws IOException {
-        final byte tmpBuffer[] = new byte[length];
+        final byte[] tmpBuffer = new byte[length];
         readFully(tmpBuffer, 0, tmpBuffer.length);
         return Long.parseLong(ArchiveUtils.toAsciiString(tmpBuffer), radix);
     }
@@ -480,7 +480,7 @@ public class CpioArchiveInputStream extends 
ArchiveInputStream implements
 
     private String readCString(final int length) throws IOException {
         // don't include trailing NUL in file name to decode
-        final byte tmpBuffer[] = new byte[length - 1];
+        final byte[] tmpBuffer = new byte[length - 1];
         readFully(tmpBuffer, 0, tmpBuffer.length);
         if (this.in.read() == -1) {
             throw new EOFException();
diff --git 
a/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveOutputStream.java
 
b/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveOutputStream.java
index 8df75c3..cb15410 100644
--- 
a/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveOutputStream.java
+++ 
b/src/main/java/org/apache/commons/compress/archivers/cpio/CpioArchiveOutputStream.java
@@ -500,7 +500,7 @@ public class CpioArchiveOutputStream extends 
ArchiveOutputStream implements
 
     private void pad(final int count) throws IOException{
         if (count > 0){
-            final byte buff[] = new byte[count];
+            final byte[] buff = new byte[count];
             out.write(buff);
             count(count);
         }
@@ -508,7 +508,7 @@ public class CpioArchiveOutputStream extends 
ArchiveOutputStream implements
 
     private void writeBinaryLong(final long number, final int length,
             final boolean swapHalfWord) throws IOException {
-        final byte tmp[] = CpioUtil.long2byteArray(number, length, 
swapHalfWord);
+        final byte[] tmp = CpioUtil.long2byteArray(number, length, 
swapHalfWord);
         out.write(tmp);
         count(tmp.length);
     }
diff --git 
a/src/main/java/org/apache/commons/compress/archivers/cpio/CpioUtil.java 
b/src/main/java/org/apache/commons/compress/archivers/cpio/CpioUtil.java
index f53ea44..234d49d 100644
--- a/src/main/java/org/apache/commons/compress/archivers/cpio/CpioUtil.java
+++ b/src/main/java/org/apache/commons/compress/archivers/cpio/CpioUtil.java
@@ -50,7 +50,7 @@ class CpioUtil {
 
         long ret = 0;
         int pos = 0;
-        final byte tmp_number[] = new byte[number.length];
+        final byte[] tmp_number = new byte[number.length];
         System.arraycopy(number, 0, tmp_number, 0, number.length);
 
         if (!swapHalfWord) {
diff --git 
a/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java 
b/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
index d331f65..5486bbb 100644
--- a/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
+++ b/src/main/java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java
@@ -881,7 +881,7 @@ public class SevenZFile implements Closeable {
         }
         final long numPackedStreams = totalInStreams - numBindPairs;
         assertFitsIntoInt("numPackedStreams", numPackedStreams);
-        final long packedStreams[] = new long[(int)numPackedStreams];
+        final long[] packedStreams = new long[(int)numPackedStreams];
         if (numPackedStreams == 1) {
             int i;
             for (i = 0; i < (int)totalInStreams; i++) {
diff --git 
a/src/main/java/org/apache/commons/compress/archivers/zip/X0017_StrongEncryptionHeader.java
 
b/src/main/java/org/apache/commons/compress/archivers/zip/X0017_StrongEncryptionHeader.java
index a9600a7..9e27771 100644
--- 
a/src/main/java/org/apache/commons/compress/archivers/zip/X0017_StrongEncryptionHeader.java
+++ 
b/src/main/java/org/apache/commons/compress/archivers/zip/X0017_StrongEncryptionHeader.java
@@ -260,16 +260,16 @@ public class X0017_StrongEncryptionHeader extends 
PKWareExtraHeader {
     private int hashSize;
 
     // encryption data
-    private byte ivData[];
-    private byte erdData[];
+    private byte[] ivData;
+    private byte[] erdData;
 
     // encryption key
-    private byte recipientKeyHash[];
-    private byte keyBlob[];
+    private byte[] recipientKeyHash;
+    private byte[] keyBlob;
 
     // password verification data
-    private byte vData[];
-    private byte vCRC32[];
+    private byte[] vData;
+    private byte[] vCRC32;
 
     /**
      * Get record count.
diff --git 
a/src/main/java/org/apache/commons/compress/compressors/bzip2/CRC.java 
b/src/main/java/org/apache/commons/compress/compressors/bzip2/CRC.java
index a20ea70..4b33be8 100644
--- a/src/main/java/org/apache/commons/compress/compressors/bzip2/CRC.java
+++ b/src/main/java/org/apache/commons/compress/compressors/bzip2/CRC.java
@@ -24,7 +24,7 @@ package org.apache.commons.compress.compressors.bzip2;
  * @NotThreadSafe
  */
 class CRC {
-    private static final int crc32Table[] = {
+    private static final int[] crc32Table = {
             0x00000000, 0x04c11db7, 0x09823b6e, 0x0d4326d9,
             0x130476dc, 0x17c56b6b, 0x1a864db2, 0x1e475005,
             0x2608edb8, 0x22c9f00f, 0x2f8ad6d6, 0x2b4bcb61,
diff --git 
a/src/test/java/org/apache/commons/compress/archivers/tar/TarUtilsTest.java 
b/src/test/java/org/apache/commons/compress/archivers/tar/TarUtilsTest.java
index 435ce50..bbb2b0f 100644
--- a/src/test/java/org/apache/commons/compress/archivers/tar/TarUtilsTest.java
+++ b/src/test/java/org/apache/commons/compress/archivers/tar/TarUtilsTest.java
@@ -218,13 +218,13 @@ public class TarUtilsTest {
         // COMPRESS-114
         final ZipEncoding enc = 
ZipEncodingHelper.getZipEncoding(CharsetNames.ISO_8859_1);
         final String s = 
"0302-0601-3\u00b1\u00b1\u00b1F06\u00b1W220\u00b1ZB\u00b1LALALA\u00b1\u00b1\u00b1\u00b1\u00b1\u00b1\u00b1\u00b1\u00b1\u00b1CAN\u00b1\u00b1DC\u00b1\u00b1\u00b104\u00b1060302\u00b1MOE.model";
-        final byte buff[] = new byte[100];
+        final byte[] buff = new byte[100];
         final int len = TarUtils.formatNameBytes(s, buff, 0, buff.length, enc);
         assertEquals(s, TarUtils.parseName(buff, 0, len, enc));
     }
 
     private void checkName(final String string) {
-        final byte buff[] = new byte[100];
+        final byte[] buff = new byte[100];
         final int len = TarUtils.formatNameBytes(string, buff, 0, buff.length);
         assertEquals(string, TarUtils.parseName(buff, 0, len));
     }
diff --git 
a/src/test/java/org/apache/commons/compress/compressors/bzip2/PythonTruncatedBzip2Test.java
 
b/src/test/java/org/apache/commons/compress/compressors/bzip2/PythonTruncatedBzip2Test.java
index 2bdd014..49f60df 100644
--- 
a/src/test/java/org/apache/commons/compress/compressors/bzip2/PythonTruncatedBzip2Test.java
+++ 
b/src/test/java/org/apache/commons/compress/compressors/bzip2/PythonTruncatedBzip2Test.java
@@ -113,7 +113,7 @@ public class PythonTruncatedBzip2Test {
     // Does not check parameters, so may fail if they are incompatible
     private static byte[] copyOfRange(final byte[] original, final int from, 
final int to) {
         final int length = to - from;
-        final byte buff[] = new byte[length];
+        final byte[] buff = new byte[length];
         System.arraycopy(original, from, buff, 0, length);
         return buff;
     }
diff --git 
a/src/test/java/org/apache/commons/compress/utils/FixedLengthBlockOutputStreamTest.java
 
b/src/test/java/org/apache/commons/compress/utils/FixedLengthBlockOutputStreamTest.java
index cfda61b..0a13de6 100644
--- 
a/src/test/java/org/apache/commons/compress/utils/FixedLengthBlockOutputStreamTest.java
+++ 
b/src/test/java/org/apache/commons/compress/utils/FixedLengthBlockOutputStreamTest.java
@@ -65,7 +65,7 @@ public class FixedLengthBlockOutputStreamTest {
         MockWritableByteChannel mock = new MockWritableByteChannel(blockSize, 
false);
         ByteArrayOutputStream bos = mock.bos;
         String text = "hello world avengers";
-        byte msg[] = text.getBytes();
+        byte[] msg = text.getBytes();
         int len = msg.length;
         try (FixedLengthBlockOutputStream out = new 
FixedLengthBlockOutputStream(mock, blockSize)) {
             for (int i = 0; i < len; i++) {
@@ -92,7 +92,7 @@ public class FixedLengthBlockOutputStreamTest {
         int blockSize = 13;
         MockWritableByteChannel mock = new MockWritableByteChannel(blockSize, 
false);
         String testString = "hello world";
-        byte msg[] = testString.getBytes();
+        byte[] msg = testString.getBytes();
         int reps = 17;
 
         try (FixedLengthBlockOutputStream out = new 
FixedLengthBlockOutputStream(mock, blockSize)) {
@@ -228,7 +228,7 @@ public class FixedLengthBlockOutputStreamTest {
         MockWritableByteChannel mock = new MockWritableByteChannel(blockSize, 
false);
 
         ByteArrayOutputStream bos = mock.bos;
-        byte msg[] = text.getBytes();
+        byte[] msg = text.getBytes();
         ByteBuffer buf = getByteBuffer(msg);
         try (FixedLengthBlockOutputStream out = new 
FixedLengthBlockOutputStream(mock, blockSize)) {
             out.write(buf);

Reply via email to