Skip to content

Edit #3086

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

Merged
merged 2 commits into from
Oct 20, 2016
Merged

Edit #3086

merged 2 commits into from
Oct 20, 2016

Conversation

iriark01
Copy link
Contributor

Just a text edit.

Started just because the line breaks were rendered badly on docs.mbed, but then I also did some proofreading.
Going to the rendered Doxy; the current link won't work on docs.mbed anyway, since it's relative to the repo
@0xc0170
Copy link
Contributor

0xc0170 commented Oct 20, 2016

@bogdanm

The core of the mbed-events library is the [EventQueue](EventQueue.h) class,
which represents a single event queue. The `EventQueue::dispatch` function
runs the queue, providing the context for executing events.
The core of the mbed-events library is the [EventQueue](https://docs.mbed.com/docs/mbed-os-api/en/mbed-os-5.2/api/classevents_1_1EventQueue.html) class, which represents a single event queue. The `EventQueue::dispatch` function runs the queue, providing the context for executing events:
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiousity, why the single line? Doesn't markdown join sequential lines? Single lines may make this harder to read/edit in terminals (not uncommon for READMEs).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

docs.mbed showed them as broken lines. Very obvious and ugly.

Copy link
Contributor

Choose a reason for hiding this comment

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

Could docs.mbed be fixed to avoid the ugliness? Where does the renderer live?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No. We will switch engines soon, and we'll see what happens then. In the meantime, no.

Copy link
Contributor

Choose a reason for hiding this comment

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

Coolio then

@geky
Copy link
Contributor

geky commented Oct 20, 2016

I like what you've done with the place

@sg- sg- merged commit d283c0f into ARMmbed:mbed-os-5.2 Oct 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants