[GitHub] [commons-compress] PeterAlfreadLee commented on issue #87: COMPRESS-124 : Add support for extracting sparse entries from tar archives

[GitBox <gi...@apache.org>]

PeterAlfreadLee commented on issue #87: COMPRESS-124 : Add support for extracting sparse entries from tar archives
URL: https://github.com/apache/commons-compress/pull/87#issuecomment-560079934
 
 
   @bodewig Sure. I'll modify my code.
   The `TarArchiveInputStream` could have another handling of `read` and `skip` when the entry is a sparse entry. This could easily implemented by calling `currEntry.isSparse()`. Then I can expand the original code with only a little modification of original code. Therefore I don't need a separate class `TarArchiveSparseInputStream`.
   I'll push my code in these two days.
   
   For the implemenation of reading the sparse entry, I'm combining the `BoundedInputStream` with the original tar file input stream and the `TarArchiveSparseZeroInputStream`. They are constructed by reading the sparse headers of `currEntry`. What do you think about this? Maybe you have some better ideas? @bodewig @tcurdt 

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services