-
Notifications
You must be signed in to change notification settings - Fork 178
Update DeepSleepLock #316
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
Update DeepSleepLock #316
Conversation
Remove content that is not relevant, add new example and update description.
@sg- can you review this? |
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.
Great stuff! Have a look at the RAII intro from CriticalSection. Probably a few things to steal and move here as well. https://github.com/ARMmbed/Handbook/pull/298/files
Add a page for the SleepManager API. This contains general sleep information, along with the sleep manager API and an example which uses it.
8560df5
to
de95a74
Compare
@sg- I updated this PR to talk about the constructor and destructor of RAII objects as #298 does. |
Make minor formatting change for proper rendering.
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.
Nice work on this. I made a minor formatting fix.
|
||
### Example | ||
|
||
[](https://os.mbed.com/users/c1728p9/code/SleepManager_Example_1/file/e85412b4147e/main.cpp) |
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.
Query: Is there any way to put this in the same teams location as the other examples?
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.
Yep, I updated this to point to that.
Update URL to use the mbed_example team for the example link rather than a personal repository.
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.
LGTM 👍
Update the DeepSleepLock page along with moving content from this page to a new sleep manager page.