On 02/23/2015 10:55 PM, Xueming Shen wrote:
If we're talking about using java.time from ZipEntry, then that's another story. I belive that VM startup does not need the conversion from DOS time to Java time in ZipEntry, but that should be checked to be sure...

I was talking about to use j.time for the dos<-> conversion, I would assume that was the original optimization you were talking about. With Claes's change, it is possible the j.u.zip code no longer needs to the dos->java time conversion during
startup.  The dos timestamp in Zip spec is indeed a j.time.LocalDateTime.

You're right. I'm thinking of new proposed code which does lazy DOS -> Java conversion. Previously (or currently, to be precise), constructing ZipEntry already triggered conversion.

Peter

Reply via email to