Skip to content

[zstd_stream] Now use ZSTD_decompressStream C API - #80

Merged
Viq111 merged 6 commits into
1.xfrom
viq111/stream2-decompression
Apr 3, 2020
Merged

[zstd_stream] Now use ZSTD_decompressStream C API#80
Viq111 merged 6 commits into
1.xfrom
viq111/stream2-decompression

Conversation

@Viq111

Copy link
Copy Markdown
Collaborator

Follow up of #79 but for the decompression side.

This:

This PR is against #79 branch not 1.x to be able to merge both at the same time (and have a smaller diff)

Before putting our compression/decompression buffers back in the pool, ensure we can't reuse them
- Move Decompression API from the (now deprecated) ZBUFF_decompressInit / ZBUFF_decompressContinue API to ZSTD_decompressStream
- Add some C passthrough methods to not have to allocated pointers
- Switch from ZBUFF private methods to ZSTD methods and structs
@Viq111
Viq111 requested a review from gmmeyerFebruary 4, 2020 16:51
@Viq111

Copy link
Copy Markdown
CollaboratorAuthor

@gmmeyer This is pretty similar to #79 if you want to take a look as well!

@Viq111Viq111 changed the title Viq111/stream2 decompression[zstd_stream] Now use ZSTD_decompressStream C APIFeb 4, 2020
@Viq111
Viq111 changed the base branch from viq111/stream2 to 1.xMarch 19, 2020 18:23
@Viq111
Viq111 merged commit 0e71ac6 into 1.xApr 3, 2020
@Viq111
Viq111 deleted the viq111/stream2-decompression branch April 3, 2020 17:51
Sign up for freeto 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

@Viq111@gmmeyer