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 2008/07/06 15:23:32 UTC

svn commit: r674299 - /commons/sandbox/compress/trunk/src/java/org/apache/commons/compress/archivers/zip/ZipEntry.java

Author: sebb
Date: Sun Jul  6 06:23:32 2008
New Revision: 674299

URL: http://svn.apache.org/viewvc?rev=674299&view=rev
Log:
clone() is not allowed to return null

Modified:
    commons/sandbox/compress/trunk/src/java/org/apache/commons/compress/archivers/zip/ZipEntry.java

Modified: commons/sandbox/compress/trunk/src/java/org/apache/commons/compress/archivers/zip/ZipEntry.java
URL: http://svn.apache.org/viewvc/commons/sandbox/compress/trunk/src/java/org/apache/commons/compress/archivers/zip/ZipEntry.java?rev=674299&r1=674298&r2=674299&view=diff
==============================================================================
--- commons/sandbox/compress/trunk/src/java/org/apache/commons/compress/archivers/zip/ZipEntry.java (original)
+++ commons/sandbox/compress/trunk/src/java/org/apache/commons/compress/archivers/zip/ZipEntry.java Sun Jul  6 06:23:32 2008
@@ -405,8 +405,7 @@
         catch( final Exception e )
         {
             // impossible as extra data is in correct format
-            e.printStackTrace();
-            return null;
+            throw new Error("Should never happen");// TODO replace with AssertionError? (1.4+)
         }
 
         entry.setInternalAttributes( getInternalAttributes() );