-
Notifications
You must be signed in to change notification settings - Fork 3k
RTOS : added empty and full functions to Mail and Queue #5158
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
Conversation
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.
Thanks for splitting your previous PR into two distinct PRs.
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputBuild failed! |
/morph test |
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputBuild failed! |
@studavekar I am not able to reproduce the failure above. It does not look related to this changeset (link errors for one toolchain for big callback test). My test command to reproduce:
I can see that there is similar error to another failure in #5220 (comment) I've noticed however another error here:
Based on the above, I assume this fails because there is no space on the device. |
Ah on a side note, @YarivCol could you extend tests for the new functions? |
Set as needs: work, any new API should add tests as requested above |
@0xc0170 It looks like tests were added. Are you happy with these tests? |
Build : SUCCESSBuild number : 20 |
@AnotherButler More New APIs |
Build : FAILUREBuild number : 86 |
/morph build |
1 similar comment
/morph build |
Build : SUCCESSBuild number : 290 Triggering tests/morph test |
Build : SUCCESSBuild number : 293 Triggering tests/morph test |
Test : SUCCESSBuild number : 132 |
Test : FAILUREBuild number : 133 |
@0xc0170 what is the status of this PR ?, do i need to fix something for it to be merged ? |
Waiting for CI to be completed. /morph build |
Looks like this didn't re-run. |
/morph test |
Test : SUCCESSBuild number : 186 |
@studavekar could you remove the invalid "AWS-CI uVisor Build & Test mbedos-5325"? |
@theotherjimmy github doesn't like changing status once merged, all status are marked as passed : https://api.github.com/repos/ARMmbed/mbed-os/statuses/e0a0615038f6cc491644ad6fc162eb4f2ab7aa41 |
That's fine. Thanks for looking at it. |
Description
Functions to allow the user to know the state of the queue or mail