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
Add platform/Span documentation. #683
Conversation
@donatieng Could you review ? |
Copy edit, mostly for active voice.
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.
I edited this for active voice. Please review it to make sure I didn't accidentally change the meaning of anything.
### Span class reference | ||
|
||
[![View code](https://www.mbed.com/embed/?type=library)](https://os-doc-builder.test.mbed.com/docs/development/mbed-os-api-doxy/class_span.html) | ||
|
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.
Please add an example.
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 @pan- - only that middle_half
thing, otherwise the doc is good!
docs/reference/api/platform/Span.md
Outdated
|
||
Span<uint8_t> first_half = str_span.first(half_size); | ||
Span<uint8_t> second_half = str_span.last(half_size); | ||
Span<uint8_t> middle_half = str_span.subspan(/* offset */ half_size / 2, half_size); |
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.
middle_half
sounds a bit weird to me
@pan- Please address the comments from @donatieng |
@pan- Question: why do all code snippets show uint8_t ? why not other bit lengths? not supported yet or these are just 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.
Just left a minor question..rest all LGTM! good work @pan- ..!
@ashok-rao Any type is supported, it is just for examples purposes. |
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..thanks @pan- ..nice one.
@donatieng I fixed your issue. |
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.
Kudos for mentioning IQR ;)
This is the associated documentation PR for ARMmbed/mbed-os#7828.