-
Notifications
You must be signed in to change notification settings - Fork 84
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
Align examples and remove reading from stdin #359
Changes from 5 commits
07c7040
cb09cff
f0a1576
7f94ce9
cc5475e
54e8596
28e4b04
0196aa7
1e3e6a6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,13 +27,17 @@ | |
#endif | ||
|
||
#define KEYEXPR "demo/example/**" | ||
#define N 10 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should align the behaviour with all other bindings, I believe we don't wait for just There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Other bindings do not implement single-threaded pub/sub examples, therefore we only need to align the configuration of these examples across their different plateform-specific implementations. In this PR, the configuration is as follows:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. As per the other comment on this file, I'll update all implementations to start with a default number of samples of There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The behaviour of a |
||
|
||
int msg_nb = 0; | ||
Check warning Code scanning / Cppcheck (reported by Codacy) misra violation 804 with no text in the supplied rule-texts-file Warning
misra violation 804 with no text in the supplied rule-texts-file
|
||
|
||
void data_handler(const z_sample_t *sample, void *ctx) { | ||
(void)(ctx); | ||
z_owned_str_t keystr = z_keyexpr_to_string(sample->keyexpr); | ||
printf(">> [Subscriber] Received ('%s': '%.*s')\n", z_loan(keystr), (int)sample->payload.len, | ||
sample->payload.start); | ||
z_drop(z_move(keystr)); | ||
msg_nb++; | ||
} | ||
|
||
void app_main(void) { | ||
|
@@ -58,7 +62,8 @@ | |
return; | ||
} | ||
|
||
while (1) { | ||
printf("Running until %d messages are received...\n", N); | ||
while (msg_nb < N) { | ||
zp_read(z_loan(s), NULL); | ||
zp_send_keep_alive(z_loan(s), NULL); | ||
zp_send_join(z_loan(s), NULL); | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this case after
10
publications the example will exit, however all other examples kepp publishing indefinitely.