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.

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

be generic about the thing to write the path into #156

Merged
merged 8 commits into from
Jul 25, 2023

Conversation

jordens
Copy link
Member

@jordens jordens commented Jul 25, 2023

The downside is that we don't check capacity anymore.
For most things that are Write they don't have a capacity.
On the other hand this was always done redundantly also in the mqtt client.

@jordens jordens changed the base branch from main to rj/serde-error July 25, 2023 13:11
@jordens jordens changed the title rj/iter path write be generic about the thing to write the path into Jul 25, 2023
@jordens jordens marked this pull request as ready for review July 25, 2023 13:13
tests/iter.rs Outdated Show resolved Hide resolved
src/lib.rs Outdated Show resolved Hide resolved
src/iter.rs Show resolved Hide resolved
Base automatically changed from rj/serde-error to main July 25, 2023 14:22
@ryan-summers ryan-summers merged commit 49a4903 into main Jul 25, 2023
8 checks passed
@ryan-summers ryan-summers deleted the rj/iter-path-write branch July 25, 2023 14:45
Sign up for free to 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