commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: svn commit: r760390 - /commons/proper/compress/trunk/src/main/java/org/apache/commons/compress/archivers/zip/ZipArchiveEntry.java
Date Tue, 31 Mar 2009 13:15:32 GMT
On 31/03/2009, sebb <sebbaz@gmail.com> wrote:
> On 31/03/2009, Stefan Bodewig <bodewig@apache.org> wrote:
>  > On 2009-03-31, <sebb@apache.org> wrote:
>  >
>  >  > URL: http://svn.apache.org/viewvc?rev=760390&view=rev
>  >  > Log:
>  >  > Set some attributes from the File
>  >
>  >
>  > >      public ZipArchiveEntry(File inputFile, String entryName) {
>  >  >          this(entryName);
>  >  >+         setSize(inputFile.length());
>  >  >+         setTime(inputFile.lastModified());
>  >  >          // TODO are there any other fields we can set here?
>  >  >      }
>  >
>  >
>  > setSize is dangerous here since it may very well > 0 for directories
>  >  on Unix like systems.
>
>
> Oops!
>
>  However it should be OK for regular files, no?

I've fixed all the ArchiveEntry constructors.

>  I think we need some test cases which check the behaviour with directories ...
>

Still to do ...

>  >  Stefan
>  >
>  >  ---------------------------------------------------------------------
>  >  To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>  >  For additional commands, e-mail: dev-help@commons.apache.org
>  >
>  >
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


Mime
View raw message