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;
 

Reply via email to