You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2013/01/23 14:02:35 UTC

svn commit: r1437373 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/jar/JarArchiveEntry.java

Author: sebb
Date: Wed Jan 23 13:02:35 2013
New Revision: 1437373

URL: http://svn.apache.org/viewvc?rev=1437373&view=rev
Log:
DOcument unexpected behaviour

Modified:
    commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/jar/JarArchiveEntry.java

Modified: commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/jar/JarArchiveEntry.java
URL: http://svn.apache.org/viewvc/commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/jar/JarArchiveEntry.java?rev=1437373&r1=1437372&r2=1437373&view=diff
==============================================================================
--- commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/jar/JarArchiveEntry.java (original)
+++ commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/jar/JarArchiveEntry.java Wed Jan 23 13:02:35 2013
@@ -32,6 +32,7 @@ import org.apache.commons.compress.archi
  */
 public class JarArchiveEntry extends ZipArchiveEntry {
 
+    // These are always null - see https://issues.apache.org/jira/browse/COMPRESS-18 for discussion
     private final Attributes manifestAttributes = null;
     private final Certificate[] certificates = null;
 
@@ -62,6 +63,11 @@ public class JarArchiveEntry extends Zip
             System.arraycopy(certificates, 0, certs, 0, certs.length);
             return certs;
         }
+        /*
+         * Note, the method
+         * Certificate[] java.util.jar.JarEntry.getCertificates()
+         * also returns null or the list of certificates (but not copied)
+         */
         return null;
     }