You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2024/12/16 21:38:19 UTC

(commons-compress) 02/03: Test for close state is already done in the super call

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

commit b8891117d12b60b7a90684c49f8fe28ce7c6d531
Author: Gary D. Gregory <ga...@gmail.com>
AuthorDate: Mon Dec 16 16:35:43 2024 -0500

    Test for close state is already done in the super call
---
 .../commons/compress/archivers/cpio/CpioArchiveOutputStream.java      | 4 +---
 .../apache/commons/compress/archivers/tar/TarArchiveOutputStream.java | 4 +---
 2 files changed, 2 insertions(+), 6 deletions(-)

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 be57b664c..b6f028e30 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
@@ -175,9 +175,7 @@ public class CpioArchiveOutputStream extends ArchiveOutputStream<CpioArchiveEntr
                 finish();
             }
         } finally {
-            if (!isClosed()) {
-                super.close();
-            }
+            super.close();
         }
     }
 
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 fec8eb2dc..cfe143cd1 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
@@ -295,9 +295,7 @@ public class TarArchiveOutputStream extends ArchiveOutputStream<TarArchiveEntry>
                 finish();
             }
         } finally {
-            if (!isClosed()) {
-                super.close();
-            }
+            super.close();
         }
     }