Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Fix CI failures #394
Fix CI failures #394
Changes from all commits
84b11ad
d356881
cb554fd
f0545eb
6e10617
29fca01
bb65bc8
39cfc82
245aade
790bd62
3bab290
2ac04a2
b5c702b
8e37ab8
cac8b3d
54c291e
a657a1d
e3707c1
a16d84c
5216d7a
b21b11d
1f45c9d
c51ffe2
d9ed7eb
1c3bfa0
9c2cf4c
95506c9
9f7925b
0d9f25b
7c724c3
2f44c57
4432e1c
2e813e7
3e42741
22bf428
8e01007
ecb919a
27b9189
3511b3d
24877da
4d5177e
8101af7
31e28a4
2269b96
2f495ad
83310d7
ade5a2b
22d35de
f46217b
d55a0d1
fc49c87
90082b4
6542757
3baef3e
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
It's safer to use
int n = ~0;
in such a way we always have the guaranteen
ismax(int)
regardless the actual size ofint
.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.
With a signed integer
~0
will be interpreted as-1
. As this is also an example thing, I suggest we fix this with the z_sleep later.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.
You are right. Technically it should be an
unsigned int
and not anint
.