Skip to content

Fix and add test for ConditionVariable #5508

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 16, 2017
Merged

Fix and add test for ConditionVariable #5508

merged 1 commit into from
Nov 16, 2017

Conversation

c1728p9
Copy link
Contributor

@c1728p9 c1728p9 commented Nov 15, 2017

Fix the circular linked list handling in ConditionVariable and add a test to validate the linked list implementation.

@c1728p9
Fix and add test for ConditionVariable
93cf15d 
Fix the circular linked list handling in ConditionVariable and add a
test to validate the linked list implementation.
@c1728p9 c1728p9 mentioned this pull request Nov 15, 2017
Closed
@c1728p9
Copy link
Contributor Author

c1728p9 commented Nov 16, 2017

/morph build

@mbed-ci
Copy link

mbed-ci commented Nov 16, 2017

Test : FAILURE

Build number : 336
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/5508/336

@mbed-ci
Copy link

mbed-ci commented Nov 16, 2017

Build : SUCCESS

Build number : 530
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5508/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@mbed-ci
Copy link

mbed-ci commented Nov 16, 2017

Exporter Build : SUCCESS

Build number : 143
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/5508/

@mbed-ci
Copy link

mbed-ci commented Nov 16, 2017

Test : SUCCESS

Build number : 337
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/5508/337

kjbracey
kjbracey approved these changes Nov 16, 2017
View reviewed changes
Copy link
Contributor

@kjbracey kjbracey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me - a sensible simplification of the list. The semi-circular thing of the original made it awkward to work on.

@0xc0170 0xc0170 merged commit 2f9cf47 into ARMmbed:master Nov 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0xc0170 0xc0170 0xc0170 approved these changes

@kjbracey kjbracey kjbracey approved these changes

Assignees
No one assigned
Labels
release-version: 5.7.0-rc1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@c1728p9 @mbed-ci @0xc0170 @kjbracey