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

Release deep sleep lock in destructor of Timer class #5095

Merged
merged 1 commit into from Sep 27, 2017

Conversation

Projects
None yet
4 participants
@c1728p9
Contributor

c1728p9 commented Sep 14, 2017

Release the deep sleep lock when running instances of the Timer class are deleted. This ensures that deep sleep locks are properly released by the Timer class.

Release deep sleep lock in destructor of Timer class
Release the deep sleep lock when running instances of the Timer class
are deleted. This ensures that deep sleep locks are properly released
by the Timer class.
@c1728p9

This comment has been minimized.

Contributor

c1728p9 commented Sep 14, 2017

/morph test

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 14, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1305

All builds and test passed!

@theotherjimmy

This comment has been minimized.

Contributor

theotherjimmy commented Sep 21, 2017

We will run nightly on this soon.

@c1728p9

This comment has been minimized.

Contributor

c1728p9 commented Sep 25, 2017

/morph test-nightly

@mbed-bot

This comment has been minimized.

mbed-bot commented Sep 26, 2017

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test-nightly

Output

mbed Build Number: 1386

All builds and test passed!

@0xc0170 0xc0170 added ready for merge and removed needs: CI labels Sep 26, 2017

@theotherjimmy theotherjimmy merged commit 8508d26 into ARMmbed:master Sep 27, 2017

5 checks passed

Cam-CI uvisor Build & Test Success
Details
ci/morph-test Job has completed
Details
ci/morph-test-nightly Job has completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment