Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

Merge push into pull functions #17

Open
tmplt opened this issue Aug 14, 2021 · 1 comment
Open

Merge push into pull functions #17

tmplt opened this issue Aug 14, 2021 · 1 comment

Comments

@tmplt
Copy link
Member

tmplt commented Aug 14, 2021

i.e., the API should change to Decoder::pull_with_timestamp(data: &[u8]) and Decoder::pull(data: &[u8]). "Pull" verb should probably be replaced with something more fitting, now that there wont be a corresponding "push" verb.
@tmplt
Copy link
Member Author

tmplt commented Aug 14, 2021

This is an involved change. pull will now have to return a Vec<Result<..>>.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tmplt