PeterAlfredLee commented on pull request #169: URL: https://github.com/apache/commons-compress/pull/169#issuecomment-787572402
> The javadoc of the new option is not completely accurate, as it will always add zip64 entries to the central directory, but not necessarily encode the disk number and relative offset. Yes, it is. I pushed a new commit with only the javadoc of the new option changed. > We must ensure we add the relative LFH offset to the Zip64 extended information entry inside the central directory even if it is not too big in the case where the disk number is too big - or we'd create a gap inside of the extra field. This is currently wrong inside master as well, unless I'm wrong. This means we should encode the relative offset and disk number at the same time - even through only eithor of them is too big. Am I right about it? > Let's discuss this on the dev list. +1. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
