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 2017/09/18 20:14:02 UTC

svn commit: r1808769 - /commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/zip/test/ZipFileObjectTestCase.java

Author: ggregory
Date: Mon Sep 18 20:14:01 2017
New Revision: 1808769

URL: http://svn.apache.org/viewvc?rev=1808769&view=rev
Log:
Add comment.

Modified:
    commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/zip/test/ZipFileObjectTestCase.java

Modified: commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/zip/test/ZipFileObjectTestCase.java
URL: http://svn.apache.org/viewvc/commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/zip/test/ZipFileObjectTestCase.java?rev=1808769&r1=1808768&r2=1808769&view=diff
==============================================================================
--- commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/zip/test/ZipFileObjectTestCase.java (original)
+++ commons/proper/vfs/trunk/commons-vfs2/src/test/java/org/apache/commons/vfs2/provider/zip/test/ZipFileObjectTestCase.java Mon Sep 18 20:14:01 2017
@@ -151,7 +151,8 @@ public class ZipFileObjectTestCase {
             inputStream1 = zipFileObject1.getContent().getInputStream();
             resolveReadAssert(zipFileObject, NESTED_FILE_2);
         }
-        // The zip file is "closed", but we read from the stream now.
+        // The zip file is "closed", but we read from the stream now, which currently fails.
+        // Solve this by counting open streams and only closing when all streams are closed?
         readAndAssert(inputStream1, "1");
         // clean up
         zipFileObject1.close();