You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by bo...@apache.org on 2011/04/19 07:24:59 UTC

svn commit: r1094854 - in /commons/proper/compress/trunk/src: main/java/org/apache/commons/compress/archivers/zip/ZipFile.java test/java/org/apache/commons/compress/archivers/zip/ZipFileTest.java

Author: bodewig
Date: Tue Apr 19 05:24:58 2011
New Revision: 1094854

URL: http://svn.apache.org/viewvc?rev=1094854&view=rev
Log:
RandomAccessFile.close() doesn't throw any exception when it is closed more than once, so the guard and synchronization are not needed

Modified:
    commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java
    commons/proper/compress/trunk/src/test/java/org/apache/commons/compress/archivers/zip/ZipFileTest.java

Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java
URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java?rev=1094854&r1=1094853&r2=1094854&view=diff
==============================================================================
--- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java (original)
+++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/ZipFile.java Tue Apr 19 05:24:58 2011
@@ -213,11 +213,9 @@ public class ZipFile {
      * Closes the archive.
      * @throws IOException if an error occurs closing the archive.
      */
-    public synchronized void close() throws IOException {
-        if (!closed) {
-            closed = true;
-            archive.close();
-        }
+    public void close() throws IOException {
+        closed = true;
+        archive.close();
     }
 
     /**

Modified: commons/proper/compress/trunk/src/test/java/org/apache/commons/compress/archivers/zip/ZipFileTest.java
URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/test/java/org/apache/commons/compress/archivers/zip/ZipFileTest.java?rev=1094854&r1=1094853&r2=1094854&view=diff
==============================================================================
--- commons/proper/compress/trunk/src/test/java/org/apache/commons/compress/archivers/zip/ZipFileTest.java (original)
+++ commons/proper/compress/trunk/src/test/java/org/apache/commons/compress/archivers/zip/ZipFileTest.java Tue Apr 19 05:24:58 2011
@@ -88,6 +88,16 @@ public class ZipFileTest extends TestCas
         assertEntryName(l, 22, "ZipUtil");
     }
 
+    public void testDoubleClose() throws Exception {
+        readOrderTest();
+        zf.close();
+        try {
+            zf.close();
+        } catch (Exception ex) {
+            fail("Caught exception of second close");
+        }
+    }
+
     /*
      * ordertest.zip has been handcrafted.
      *