Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Additional handling of Files in TarEntries #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

phillipjohnson
Copy link

I was using your library today and there was some functionality that I felt would make the library a bit easier to use. Thanks for your effort in creating this utility, it doesn't seem like there is much else out there except for the bloated Apache library. The commit changes below are pretty self-explanatory. Let me know what you think.

  • When iterating over the tar stream, each Entry will get a reference to its associated File.
  • Added a utility method for easily retrieving all of the TarEntries in a given archive.
  • Updated/added unit tests accordingly.

Note that some small tweaks were required for the unit tests because the bytesRead count now must also include the bytes of the file just read.

When iterating over the tar stream, each Entry will get a reference to
the associated File.

Added a utility method for easily retrieving all of the TarEntries in a
given archive.

Updated/added unit tests accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants