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.

Sign up for GitHub

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

Jump to bottom

StreamCopier range support #4413

Closed
aleks-f opened this issue Jan 23, 2024 · 2 comments · Fixed by #4474
Closed

StreamCopier range support #4413

aleks-f opened this issue Jan 23, 2024 · 2 comments · Fixed by #4474
Assignees
@pavledragisic
Labels
feature fixed
Milestone
Release 1.14.0

Comments

@aleks-f
Copy link
Member

aleks-f commented Jan 23, 2024

Currently in Poco there is StreamCopier::copyStream static method which copy one stream to another.
If will be useful to have method to copy part(specified range) of one stream to another.
@aleks-f aleks-f added this to the Release 1.14.0 milestone Jan 23, 2024
@aleks-f aleks-f added this to 1.14 Jan 23, 2024
pavledragisic pushed a commit that referenced this issue Feb 27, 2024
feat(Foundation): StreamCopier copyStreamRange #4413
6528a46
@pavledragisic pavledragisic moved this to In Progress in 1.14 Feb 27, 2024
@pavledragisic pavledragisic linked a pull request Feb 27, 2024 that will close this issue
Add copyStreamRange to StreamCopier #4474
Merged
aleks-f added a commit that referenced this issue Feb 28, 2024
@pavledragisic @aleks-f
Add copyStreamRange to StreamCopier (#4474)
8119259 
* feat(Foundation): StreamCopier copyStreamRange #4413

* chore(ci): disable instalation of unused databases in odbc job

---------

Co-authored-by: Pavle <[email protected]>
Co-authored-by: Alex Fabijanic <[email protected]>
@github-project-automation github-project-automation bot moved this from In Progress to Done in 1.14 Feb 28, 2024
@aleks-f aleks-f added the fixed label Feb 28, 2024
@aleks-f
Copy link
Member Author

aleks-f commented Jul 18, 2024

seekg position should be relative to beginning:

poco/Foundation/include/Poco/StreamCopier.h

Line 147 in ed181d9

istr.seekg(rangeStart);

aleks-f added a commit that referenced this issue Jul 18, 2024
@aleks-f
seekg position should be relative to beginning #4413 aleph-us/devs#397
1edabc1
@aleks-f
Copy link
Member Author

aleks-f commented Jul 18, 2024

linked issue was wrong, correct issue: https://github.com/aleph-us/devs/issues/435

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pavledragisic pavledragisic

Labels
feature fixed
Projects
1.14
Status: Done
Milestone
Release 1.14.0
Development

Successfully merging a pull request may close this issue.

Add copyStreamRange to StreamCopier pocoproject/poco
2 participants
@aleks-f @pavledragisic