Skip to content

[Docs] Added useStaticBuffers function description #5820

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
Nov 4, 2021

Conversation

pedrominatel
Copy link
Member

Summary

Added useStaticBuffers function description on the documentation.

Impact

None.

@pedrominatel pedrominatel added the Type: Documentation Issue pertains to Documentation of Arduino ESP32 label Oct 29, 2021
@pedrominatel pedrominatel added this to the 2.0.1 milestone Oct 29, 2021
@pedrominatel pedrominatel self-assigned this Oct 29, 2021
@pedrominatel pedrominatel changed the title [Docs]: Added useStaticBuffers function description [Docs] Added useStaticBuffers function description Oct 29, 2021
The use of dynamic allocation is recommended to save memory and reduce resources usage. However, the dynamic performs slightly slower than the static allocation.
Use static allocation if you want to have more performance and if your application is multi-tasking.

By default, the memory allocation will be set to *dynamic* if this function is not being used.
Copy link
Contributor

Choose a reason for hiding this comment

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

What about highlighting this default setting with bold or box format?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done! PTAL

Copy link
Contributor

Choose a reason for hiding this comment

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

I will probably put into bold the whole sentence but this is better than nothing :) other looks good to me!

@VojtechBartoska VojtechBartoska self-requested a review November 2, 2021 12:49
@me-no-dev me-no-dev merged commit 3e851b5 into espressif:master Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Documentation Issue pertains to Documentation of Arduino ESP32
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants