introduce query channels #222
Conversation
|
It generally looks good to me. However, can you add an example on how to use |
|
It seems there is a merge conflict on |
| */ | ||
| typedef struct z_owned_query_t { | ||
| void *_0; | ||
| } z_owned_query_t; |
There was a problem hiding this comment.
As per-naming convention, z_x means that is also available in zenoh-pico. However, this feature will initially only available in zenoh-c... should it (and all related functions) be called zc_?
There was a problem hiding this comment.
The plan was originally to introduce these symbols in pico during the release... They should arrive soon, but we could rename these for now until pico has them too.
There was a problem hiding this comment.
The rename would introduce a small break in zenoh-cpp, which would be a thorn in the release's side, so let's keep the z_ prefix, with the goal of having pico finish up implementation by mid-January.
There was a problem hiding this comment.
Let's keep the name then to avoid excessive back-and-forth.
Since owned queries now exist, channels to allow moving them around would be useful.