theobisproject opened a new pull request #113:
URL: https://github.com/apache/commons-compress/pull/113


   Basic idea behind the implementation is to read all headers at instantiation 
and save the position to the entry data. This will then be accessed as a random 
access file when reading the entry.
   I tried to touch as less exisiting code as possible and only did refactor 
existing code were I felt it was really needed. 
   
   I have the following TODOs left in the code where I think your opinion is 
needed
   
   - TarArchiveInputStream#canReadEntryData: as noted in the TODO sparse 
entries are now supported. I see no need for this function any longer and would 
not include it in the TarFile api. Also we should update the implementation to 
reflect the now supported sparse entries.
   - In the SparseFilesTest extracting of sparsefile-0.1 in pax_gnu_sparse.tar 
is skipped because of problems with tar on Ubuntu 16.04. I tested this on 
Ubuntu 18.04 and everything worked fine. Should we reenable extracting of the 
file?


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


Reply via email to