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

Draft: add read implementation instead of function #22

Draft

torfmaster wants to merge 3 commits into main from read-interface

Owner

torfmaster commented Apr 3, 2022

An attempt to solve #18. It seems to be a feasible goal. Further steps

improve docs and make the Read implementation the preferred way
also apply the concepts also for decompression. Open question: how to deal with domain specific errors? Logging? Create extra recovering/analysis related methods for decompression?
attempt Write base trait implementations as well to get full compatibility

Also fyi @caglaryucekaya.


          add read implementation instead of function

f36cdaf

torfmaster force-pushed the read-interface branch from 6c593c5 to f36cdaf Compare

April 3, 2022 19:02

torfmaster added 2 commits

April 4, 2022 09:08


          fix doctest

021eff3


          add original encode_stream as a deprecated method

2be5696

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

Labels

None yet