You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by bo...@apache.org on 2021/05/22 13:25:01 UTC
[commons-compress] branch master updated: COMPRESS-580 - Remove
redundant operation
This is an automated email from the ASF dual-hosted git repository.
bodewig 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 ce1c3f0 COMPRESS-580 - Remove redundant operation
new 93aab64 Merge pull request #196 from arturobernalg/feature/COMPRESS-580
ce1c3f0 is described below
commit ce1c3f0d6d3ef873d43fa1337821800de7cc77ce
Author: Arturo Bernal <ar...@gmail.com>
AuthorDate: Sat May 22 14:36:16 2021 +0200
COMPRESS-580 - Remove redundant operation
---
.../archivers/arj/ArjArchiveInputStream.java | 4 +--
.../commons/compress/archivers/sevenz/CLI.java | 2 +-
.../compress/archivers/sevenz/SevenZFile.java | 4 +--
.../compress/archivers/zip/ExtraFieldUtils.java | 4 +--
.../commons/compress/archivers/zip/ZipFile.java | 2 +-
.../apache/commons/compress/ArchiveUtilsTest.java | 2 +-
.../compress/archivers/cpio/CpioArchiveTest.java | 6 ++--
.../lz4/BlockLZ4CompressorRoundtripTest.java | 6 ++--
.../lz4/FramedLZ4CompressorRoundtripTest.java | 36 ++++++++++------------
.../compressors/z/ZCompressorInputStreamTest.java | 2 +-
10 files changed, 31 insertions(+), 37 deletions(-)
diff --git a/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java b/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java
index 8a420c0..b0c16b2 100644
--- a/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java
+++ b/src/main/java/org/apache/commons/compress/archivers/arj/ArjArchiveInputStream.java
@@ -115,10 +115,10 @@ public class ArjArchiveInputStream extends ArchiveInputStream {
buffer.write(nextByte);
}
if (charsetName != null) {
- return new String(buffer.toByteArray(), charsetName);
+ return buffer.toString(charsetName);
}
// intentionally using the default encoding as that's the contract for a null charsetName
- return new String(buffer.toByteArray());
+ return buffer.toString();
}
}
diff --git a/src/main/java/org/apache/commons/compress/archivers/sevenz/CLI.java b/src/main/java/org/apache/commons/compress/archivers/sevenz/CLI.java
index 0d1510d..dfa1c58 100644
--- a/src/main/java/org/apache/commons/compress/archivers/sevenz/CLI.java
+++ b/src/main/java/org/apache/commons/compress/archivers/sevenz/CLI.java
@@ -42,7 +42,7 @@ public class CLI {
if (!entry.isDirectory()) {
System.out.println(" " + getContentMethods(entry));
} else {
- System.out.println("");
+ System.out.println();
}
}
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 39d8b82..b0b5abf 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
@@ -178,7 +178,7 @@ public class SevenZFile implements Closeable {
* @since 1.19
*/
public SevenZFile(final SeekableByteChannel channel, final SevenZFileOptions options) throws IOException {
- this(channel, DEFAULT_FILE_NAME, (char[]) null, options);
+ this(channel, DEFAULT_FILE_NAME, null, options);
}
/**
@@ -371,7 +371,7 @@ public class SevenZFile implements Closeable {
* @since 1.19
*/
public SevenZFile(final File fileName, final SevenZFileOptions options) throws IOException {
- this(fileName, (char[]) null, options);
+ this(fileName, null, options);
}
/**
diff --git a/src/main/java/org/apache/commons/compress/archivers/zip/ExtraFieldUtils.java b/src/main/java/org/apache/commons/compress/archivers/zip/ExtraFieldUtils.java
index 84e769d..8908628 100644
--- a/src/main/java/org/apache/commons/compress/archivers/zip/ExtraFieldUtils.java
+++ b/src/main/java/org/apache/commons/compress/archivers/zip/ExtraFieldUtils.java
@@ -68,11 +68,11 @@ public class ExtraFieldUtils {
final ZipExtraField ze = (ZipExtraField) c.newInstance();
implementations.put(ze.getHeaderId(), c);
} catch (final ClassCastException cc) { // NOSONAR
- throw new RuntimeException(c + " doesn\'t implement ZipExtraField"); //NOSONAR
+ throw new RuntimeException(c + " doesn't implement ZipExtraField"); //NOSONAR
} catch (final InstantiationException ie) { // NOSONAR
throw new RuntimeException(c + " is not a concrete class"); //NOSONAR
} catch (final IllegalAccessException ie) { // NOSONAR
- throw new RuntimeException(c + "\'s no-arg constructor is not public"); //NOSONAR
+ throw new RuntimeException(c + "'s no-arg constructor is not public"); //NOSONAR
}
}
diff --git a/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java b/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java
index a3a65a4..8792563 100644
--- a/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java
+++ b/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java
@@ -475,7 +475,7 @@ public class ZipFile implements Closeable {
public Iterable<ZipArchiveEntry> getEntries(final String name) {
final List<ZipArchiveEntry> entriesOfThatName = nameMap.get(name);
return entriesOfThatName != null ? entriesOfThatName
- : Collections.<ZipArchiveEntry>emptyList();
+ : Collections.emptyList();
}
/**
diff --git a/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java b/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java
index 5ee9a43..ebec4f2 100644
--- a/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java
+++ b/src/test/java/org/apache/commons/compress/ArchiveUtilsTest.java
@@ -133,7 +133,7 @@ public class ArchiveUtilsTest extends AbstractTestCase {
@Test
public void testIsEqual() {
- assertTrue(ArchiveUtils.isEqual((byte[]) null, 0, 0, (byte[]) null, 0, 0));
+ assertTrue(ArchiveUtils.isEqual(null, 0, 0, null, 0, 0));
}
diff --git a/src/test/java/org/apache/commons/compress/archivers/cpio/CpioArchiveTest.java b/src/test/java/org/apache/commons/compress/archivers/cpio/CpioArchiveTest.java
index 128289b..dc76db4 100644
--- a/src/test/java/org/apache/commons/compress/archivers/cpio/CpioArchiveTest.java
+++ b/src/test/java/org/apache/commons/compress/archivers/cpio/CpioArchiveTest.java
@@ -34,12 +34,10 @@ public class CpioArchiveTest {
@Parameters(name = "using {0}")
public static Collection<Object[]> factory() {
- return Arrays.asList(new Object[][] {
- new Object[] { CpioConstants.FORMAT_NEW },
+ return Arrays.asList(new Object[] { CpioConstants.FORMAT_NEW },
new Object[] { CpioConstants.FORMAT_NEW_CRC },
new Object[] { CpioConstants.FORMAT_OLD_ASCII },
- new Object[] { CpioConstants.FORMAT_OLD_BINARY },
- });
+ new Object[] { CpioConstants.FORMAT_OLD_BINARY });
}
private final short format;
diff --git a/src/test/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorRoundtripTest.java b/src/test/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorRoundtripTest.java
index a36d8e0..49595b4 100644
--- a/src/test/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorRoundtripTest.java
+++ b/src/test/java/org/apache/commons/compress/compressors/lz4/BlockLZ4CompressorRoundtripTest.java
@@ -38,13 +38,11 @@ public final class BlockLZ4CompressorRoundtripTest extends AbstractTestCase {
@org.junit.runners.Parameterized.Parameters(name = "using {0}")
public static Collection<Object[]> factory() {
- return Arrays.asList(new Object[][] {
- new Object[] { "default", BlockLZ4CompressorOutputStream.createParameterBuilder().build() },
+ return Arrays.asList(new Object[] { "default", BlockLZ4CompressorOutputStream.createParameterBuilder().build() },
new Object[] { "tuned for speed",
BlockLZ4CompressorOutputStream.createParameterBuilder().tunedForSpeed().build() },
new Object[] { "tuned for compression ratio",
- BlockLZ4CompressorOutputStream.createParameterBuilder().tunedForCompressionRatio().build() }
- });
+ BlockLZ4CompressorOutputStream.createParameterBuilder().tunedForCompressionRatio().build() });
}
private final String config;
diff --git a/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorRoundtripTest.java b/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorRoundtripTest.java
index dbf0458..4d35d99 100644
--- a/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorRoundtripTest.java
+++ b/src/test/java/org/apache/commons/compress/compressors/lz4/FramedLZ4CompressorRoundtripTest.java
@@ -39,25 +39,23 @@ public final class FramedLZ4CompressorRoundtripTest extends AbstractTestCase {
@Parameters(name = "using {0}")
public static Collection<Object[]> factory() {
- return Arrays.asList(new Object[][] {
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.K64) },
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.K256) },
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M1) },
- new Object[] { FramedLZ4CompressorOutputStream.Parameters.DEFAULT },
- // default without content checksum
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M4,
- false, false, false) },
- // default with block checksum
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M4,
- true, true, false) },
- // small blocksize (so we get enough blocks) and enabled block dependency, otherwise defaults
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.K64,
- true, false, true) },
- // default, tuned for speed
- new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M4,
- true, false, false, BlockLZ4CompressorOutputStream.createParameterBuilder()
- .tunedForSpeed().build()) },
- });
+ return Arrays.asList(new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.K64) },
+ new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.K256) },
+ new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M1) },
+ new Object[] { FramedLZ4CompressorOutputStream.Parameters.DEFAULT },
+ // default without content checksum
+ new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M4,
+ false, false, false) },
+ // default with block checksum
+ new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M4,
+ true, true, false) },
+ // small blocksize (so we get enough blocks) and enabled block dependency, otherwise defaults
+ new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.K64,
+ true, false, true) },
+ // default, tuned for speed
+ new Object[] { new FramedLZ4CompressorOutputStream.Parameters(FramedLZ4CompressorOutputStream.BlockSize.M4,
+ true, false, false, BlockLZ4CompressorOutputStream.createParameterBuilder()
+ .tunedForSpeed().build()) });
}
private final FramedLZ4CompressorOutputStream.Parameters params;
diff --git a/src/test/java/org/apache/commons/compress/compressors/z/ZCompressorInputStreamTest.java b/src/test/java/org/apache/commons/compress/compressors/z/ZCompressorInputStreamTest.java
index 5935173..6b28d0e 100644
--- a/src/test/java/org/apache/commons/compress/compressors/z/ZCompressorInputStreamTest.java
+++ b/src/test/java/org/apache/commons/compress/compressors/z/ZCompressorInputStreamTest.java
@@ -42,7 +42,7 @@ public class ZCompressorInputStreamTest {
@Test(expected = IOException.class)
public void testFailsToCreateZCompressorInputStreamAndThrowsIOException() throws IOException {
- final SequenceInputStream sequenceInputStream = new SequenceInputStream(Collections.<InputStream>emptyEnumeration());
+ final SequenceInputStream sequenceInputStream = new SequenceInputStream(Collections.emptyEnumeration());
final ZCompressorInputStream zCompressorInputStream = new ZCompressorInputStream(sequenceInputStream);
}