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

Added callbacks to ArchiveFile.Extract and Entry.Extract for progress events. #60

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

runemoennike
Copy link

The IArchiveExtractCallback contains SetTotal and SetCompleted which are called at various points during file extraction. These points differ between file formats, but along with GetStream some form of progress reporting is possible, which is what this commit introduces.

The Extract methods on ArchiveFile and Entry now accept an event handler that will be called with the new ArchiveExtractionProgressEventArgs and EntryExtractionProgressEventArgs, respectively, as the extraction progresses. Please note that it is very inconsistent when this is invoked, depending on the format. However, it is guaranteed to be called at least at the beginning and end of extracting each file.

@runemoennike runemoennike mentioned this pull request Aug 24, 2022
@adoconnection
Copy link
Owner

Rune, thanks for your work!
I'll review asap, clients are tearing me apart 😅

@runemoennike
Copy link
Author

No worries :)

@SalmaBegumJSR
Copy link

@runemoennike
how can i use your update, i downloaded the latest source but that does not includes your files/changes :(
and more over i am not that good on git..

is it possible for you to give me the updates source and a test function that can monitor the progress please?

thanks in advance @runemoennike @adoconnection

best regards

@runemoennike
Copy link
Author

runemoennike commented Jan 11, 2023

@SalmaBegumJSR you have to clone my fork of the repo, and switch to branch 55. If you do it from a terminal/console, it looks like this:

git clone https://github.com/runemoennike/SevenZipExtractor
cd SevenZipExtractor
git checkout 55

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.

3 participants