Skip to content

add set_stream to DecompIO #11183

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
Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

add set_stream to DecompIO #11183

wants to merge 5 commits into from
+73 −14

Conversation

keredson
Copy link

@keredson keredson commented Apr 2, 2023

This adds a new function to DecompIO to change the stream being read from. It's similar to creating a new DecompIO, but reuses the existing 32k buffer. This is useful for processing large numbers of zlib streams (like what a git client does during a clone operation).

Feature discussed here: #11146

@github-actions
Copy link

github-actions bot commented Apr 2, 2023

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:  +448 +0.056% standard[incl +32(data)]
      stm32:  +228 +0.058% PYBV10
        rp2:  +240 +0.074% PICO

@dpgeorge dpgeorge added the extmod Relates to extmod/ directory in source label Apr 11, 2023
This was referenced Feb 29, 2024
Merged
Closed
@projectgus
Copy link
Contributor

This is an automated heads-up that we've just merged a Pull Request
that removes the STATIC macro from MicroPython's C API.

See #13763

A search suggests this PR might apply the STATIC macro to some C code. If it
does, then next time you rebase the PR (or merge from master) then you should
please replace all the STATIC keywords with static.

Although this is an automated message, feel free to @-reply to me directly if
you have any questions about this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
extmod Relates to extmod/ directory in source
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@keredson @projectgus @dpgeorge