RFR 6303183: Support NTFS and Unix-style timestamps for entries in Zip files

Xueming Shen xueming.shen at oracle.com
Wed May 29 22:17:35 UTC 2013


Hi Martin,

Somehow most of your comments are "blank lines" in my mailbox, just wonder if
you can send them again. Based on the "hints", I updated those javadoc comments
in ZipUtils to "**".

The code has been updated slightly (mainly to not output the extra time stamp if
it already exists in the passed in/read in/user defined "extra' field. I added a test
TestExtraTime to test couple use scenario and also updated couple existing tests
(which depend on the hard coded extra field size...).

Now all unit/regression tests are passed.

http://cr.openjdk.java.net/~sherman/6303183/webrev/

As we discussed in other thread, I will continue to see the possibility of adding
atime/ctime support and better time granularity (via NTFS extra field).

Thanks!
-Sherman

On 05/28/2013 08:36 PM, Martin Buchholz wrote:
> Looks good to me.
>
> ---
> While you're moving code around, you could fix the typo below and change method comments to javadoc style ('*' => '**'')
> accidnetly
> and




More information about the core-libs-dev mailing list