commit f7726c4f727dd78f616374f87ae1e9424b41af67
author Stefan Bodewig <bodewig@apache.org> Fri Sep 07 09:46:05 2018 +0200
committer Stefan Bodewig <bodewig@apache.org> Fri Sep 07 09:46:05 2018 +0200
tree 46236eaedd9fd24a8f2c4417f8e2fd5d5a83a47f
parent 5fd497fa380c84d74718ef1b0c0b39b31a4fae30

correctly set hasCo..LevelChanged with repeated setLevel calls

see Ant bug https://bz.apache.org/bugzilla/show_bug.cgi?id=62686

src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveOutputStream.java

diff --git a/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveOutputStream.java b/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveOutputStream.java
index 584bb46..76aa0e1 100644
--- a/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveOutputStream.java
+++ b/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveOutputStream.java
@@ -875,7 +875,10 @@
             throw new IllegalArgumentException("Invalid compression level: "
                                                + level);
         }
-        hasCompressionLevelChanged = (this.level != level);
+        if (this.level == level) {
+            return;
+        }
+        hasCompressionLevelChanged = true;
         this.level = level;
     }