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

write::FrameDecoder #48

Open
Icelk opened this issue May 24, 2022 · 4 comments · May be fixed by #50
Open

write::FrameDecoder #48

Icelk opened this issue May 24, 2022 · 4 comments · May be fixed by #50
Labels
enhancement

Comments

@Icelk
Copy link

Icelk commented May 24, 2022

Hi!
What is the status of write::FrameDecoder?
I would be very grateful if you implemented it, as it could reduce a large allocation in my code.
Thanks for the great pure-rust compression algorithm!
@BurntSushi
Copy link
Owner

There is no status.

I have no plans to work on this. PRs are welcome.

@Icelk
Copy link
Author

Icelk commented May 24, 2022

I'll work on it.

@BurntSushi
Copy link
Owner

Thanks.

I noticed you filed another issue too.

Please be aware that I haven't worked on this crate in a long time, and thus lack a lot of context that would be too costly for me to re-acquire. I just don't have the time. So if you can, please do try to include as much context as you can when asking questions or submitting patches.

@Icelk
Copy link
Author

Icelk commented May 24, 2022

I will be sure to add as much context as possible to make the review pain-free.

@Icelk Icelk linked a pull request May 24, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Write framedecoder Icelk/rust-snappy
2 participants
@BurntSushi @Icelk