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

KV: add add/update functions, unit tests #337

Merged
merged 9 commits into from
Sep 14, 2023
Merged

KV: add add/update functions, unit tests #337

merged 9 commits into from
Sep 14, 2023

Conversation

bradjc
Copy link
Contributor

@bradjc bradjc commented Aug 16, 2023

Improvements to the KV support to match the kernel.

ppannuto
ppannuto previously approved these changes Aug 16, 2023
examples/tests/kv/main.c Outdated Show resolved Hide resolved
ppannuto
ppannuto previously approved these changes Aug 16, 2023
printf("Added key-value\n");
}

} else if (strncmp(read_buf, "update", 3) == 0) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be a 7?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@bradjc
Copy link
Contributor Author

bradjc commented Sep 9, 2023

Merge this?

@ppannuto ppannuto added this pull request to the merge queue Sep 14, 2023
Merged via the queue into master with commit f6684f4 Sep 14, 2023
2 checks passed
@ppannuto ppannuto deleted the kv-addupdate branch September 14, 2023 17:46
tyler-potyondy pushed a commit to tyler-potyondy/libtock-c that referenced this pull request Mar 13, 2024
KV: add add/update functions, unit tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants