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

Provide result_handler for send_data/audio_frame/video_frame #309

Closed
sunxilin opened this issue Nov 21, 2024 · 1 comment · Fixed by #341
Closed

Provide result_handler for send_data/audio_frame/video_frame #309

sunxilin opened this issue Nov 21, 2024 · 1 comment · Fixed by #341
Assignees

Comments

@sunxilin
Copy link
Collaborator

No description provided.

@sunxilin sunxilin self-assigned this Nov 21, 2024
@sunxilin
Copy link
Collaborator Author

send_xxx 是异步语义
=> C send_xxx 有 callback, 参数是 error. cmd 那条路的 callback 额外有 cmd_result 的参数.
=> C++ send_xxx 跟 C 一样
=> go send_xxx 跟 C 一样, 有 callback
=> python (non-asyncio) send_xxx 跟 C 一样, 有 callback
=> python (asyncio) send_xxx 没有 callback, 返回值是 error 变量, cmd 那条路的返回值多一个 cmd_result (error 要不要还是用 exception 待讨论)

get/set_property 是同步语义
=> C get/set_property 没有 callback, get 直接返回答案, set 返回 bool, 代表成功与否. get/set_property 都有一个 error 参数代表错误或者更多错误资讯.
=> C++ get/set_property 跟 C 一样
=> go get/set_property 跟 C 一样
=> python get/set_property (non-asyncio) 跟 C 一样
=> python get/set_property (asyncio) 跟 C 一样

==============================
其他改动:
拔掉 C++ 的 get/set_property_async 这类 API.

@sunxilin sunxilin linked a pull request Nov 27, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this to Done in Kanban Nov 28, 2024
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.

1 participant