This is an automated email from the ASF dual-hosted git repository. ggregory 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 7e5c489 Remove use of redundant String.substring() parameter. 7e5c489 is described below commit 7e5c4893364b77c3819e1057410e0ebd96d8e6f5 Author: Gary Gregory <garydgreg...@gmail.com> AuthorDate: Sun Feb 28 09:51:21 2021 -0500 Remove use of redundant String.substring() parameter. Remove unused imports. --- src/test/java/org/apache/commons/compress/archivers/LongPathTest.java | 2 +- .../java/org/apache/commons/compress/archivers/LongSymLinkTest.java | 2 +- .../org/apache/commons/compress/archivers/tar/TarArchiveEntryTest.java | 1 - .../commons/compress/archivers/tar/TarArchiveInputStreamTest.java | 2 -- 4 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/test/java/org/apache/commons/compress/archivers/LongPathTest.java b/src/test/java/org/apache/commons/compress/archivers/LongPathTest.java index bc76795..384cd91 100644 --- a/src/test/java/org/apache/commons/compress/archivers/LongPathTest.java +++ b/src/test/java/org/apache/commons/compress/archivers/LongPathTest.java @@ -141,7 +141,7 @@ public class LongPathTest extends AbstractTestCase { if (!ent.endsWith("/")) {// not a directory final int lastSlash = ent.lastIndexOf('/'); if (lastSlash >= 0) { // extract path name - expected.add(ent.substring(lastSlash + 1, ent.length())); + expected.add(ent.substring(lastSlash + 1)); } else { expected.add(ent); } diff --git a/src/test/java/org/apache/commons/compress/archivers/LongSymLinkTest.java b/src/test/java/org/apache/commons/compress/archivers/LongSymLinkTest.java index a0a6f6f..bb5a55e 100644 --- a/src/test/java/org/apache/commons/compress/archivers/LongSymLinkTest.java +++ b/src/test/java/org/apache/commons/compress/archivers/LongSymLinkTest.java @@ -140,7 +140,7 @@ public class LongSymLinkTest extends AbstractTestCase { if (!ent.endsWith("/")) {// not a directory final int lastSlash = ent.lastIndexOf('/'); if (lastSlash >= 0) { // extract path name - expected.add(ent.substring(lastSlash + 1, ent.length())); + expected.add(ent.substring(lastSlash + 1)); } else { expected.add(ent); } diff --git a/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveEntryTest.java b/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveEntryTest.java index 9a95599..6e192d2 100644 --- a/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveEntryTest.java +++ b/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveEntryTest.java @@ -37,7 +37,6 @@ import java.io.IOException; import java.nio.charset.StandardCharsets; import java.util.Locale; import org.apache.commons.compress.AbstractTestCase; -import org.apache.commons.compress.archivers.zip.ZipEncoding; import org.apache.commons.compress.archivers.zip.ZipEncodingHelper; import org.apache.commons.compress.utils.CharsetNames; import org.junit.Test; diff --git a/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStreamTest.java b/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStreamTest.java index b86c835..c95fb6f 100644 --- a/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStreamTest.java +++ b/src/test/java/org/apache/commons/compress/archivers/tar/TarArchiveInputStreamTest.java @@ -33,10 +33,8 @@ import java.io.FileInputStream; import java.io.FileOutputStream; import java.io.IOException; import java.io.InputStream; -import java.nio.charset.StandardCharsets; import java.util.Calendar; import java.util.Date; -import java.util.Map; import java.util.TimeZone; import java.util.zip.GZIPInputStream;