-
Notifications
You must be signed in to change notification settings - Fork 2k
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
tests: xtimer_msg_receive_timeout: initialize values #5216
Conversation
Looks valid to me, but was there anything that needed also to be documented, like some timer members have to be initialized before usage or so? |
Any problem if I amend it to this PR? |
not at all |
just making sure we don't forget anything that came up in the discussion in #4372 |
Provided the docfix. |
i am good with this, @kaspar030: could you please verify! |
@@ -180,7 +180,9 @@ void xtimer_usleep_until(uint32_t *last_wakeup, uint32_t usecs); | |||
* The mesage struct specified by msg parameter will not be copied, e.g., it | |||
* needs to point to valid memory until the message has been delivered. | |||
* | |||
* @param[in] timer timer struct to work with | |||
* @param[in] timer timer struct to work with. | |||
* Its xtimer_t::target and xtimer_t::long_target |
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.
::
!? ;)
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.
Auto-generates references to the fields ;-)
Or we just always initialize to zero for those functions? |
Didn't we came to the conclusion that auto-delete wouldn't work then? |
Yes. Let's go with docs for now. ACK. |
Fixes #4372