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

Remove ten_raw_msg_get_one_field_from_json and ten_raw_msg_put_one_field_to_json, and use the ten_value_t version instead #226

Closed
halajohn opened this issue Nov 2, 2024 · 1 comment · Fixed by #233
Assignees

Comments

@halajohn
Copy link
Member

halajohn commented Nov 2, 2024

TEN_RUNTIME_PRIVATE_API bool ten_raw_msg_get_one_field_from_json(
ten_msg_t *self, ten_msg_field_process_data_t *field, void *user_data,
ten_error_t *err);

TEN_RUNTIME_PRIVATE_API bool ten_raw_msg_put_one_field_to_json(
ten_msg_t *self, ten_msg_field_process_data_t *field, void *user_data,
ten_error_t *err);

@sunxilin
Copy link
Collaborator

sunxilin commented Nov 4, 2024

All types of messages traverse all fields through their respective loop_all_fields methods. By combining the two methods ten_raw_msg_put_one_field_to_json and ten_raw_msg_get_one_field_from_json, JSON serialization and deserialization can be achieved. Therefore, these two methods cannot be removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants