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
