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 34bcf90  Use isEmpty().
34bcf90 is described below

commit 34bcf90a47dba7bc7bcd9cd959007daa197fd785
Author: Gary Gregory <[email protected]>
AuthorDate: Sat Jan 16 22:10:04 2021 -0500

    Use isEmpty().
---
 .../java/org/apache/commons/compress/archivers/sevenz/SevenZFile.java   | 2 +-
 .../apache/commons/compress/archivers/tar/TarArchiveOutputStream.java   | 2 +-
 src/test/java/org/apache/commons/compress/AbstractTestCase.java         | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

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 666caab..a3cd53f 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
@@ -1342,7 +1342,7 @@ public class SevenZFile implements Closeable {
      */
     private boolean hasCurrentEntryBeenRead() {
         boolean hasCurrentEntryBeenRead = false;
-        if (deferredBlockStreams.size() > 0) {
+        if (!deferredBlockStreams.isEmpty()) {
             final InputStream currentEntryInputStream = 
deferredBlockStreams.get(deferredBlockStreams.size() - 1);
             // get the bytes remaining to read, and compare it with the size of
             // the file to figure out if the file has been read
diff --git 
a/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
 
b/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
index c5cff5e..ee84a66 100644
--- 
a/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
+++ 
b/src/main/java/org/apache/commons/compress/archivers/tar/TarArchiveOutputStream.java
@@ -389,7 +389,7 @@ public class TarArchiveOutputStream extends 
ArchiveOutputStream {
             }
             paxHeaders.putAll(entry.getExtraPaxHeaders());
 
-            if (paxHeaders.size() > 0) {
+            if (!paxHeaders.isEmpty()) {
                 writePaxHeaders(entry, entryName, paxHeaders);
             }
 
diff --git a/src/test/java/org/apache/commons/compress/AbstractTestCase.java 
b/src/test/java/org/apache/commons/compress/AbstractTestCase.java
index f419fac..2ba0a52 100644
--- a/src/test/java/org/apache/commons/compress/AbstractTestCase.java
+++ b/src/test/java/org/apache/commons/compress/AbstractTestCase.java
@@ -350,7 +350,7 @@ public abstract class AbstractTestCase {
                 }
             }
             in.close();
-            if (expected != null && expected.size() > 0) {
+            if (expected != null && !expected.isEmpty()) {
                 fail(expected.size() + " missing entries: " + 
Arrays.toString(expected.toArray()));
             }
             if (expected != null) {

Reply via email to