Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Documentation incorrectly states that C++ new and delete operators are unsupported #43636

Closed
waqas-mazhar opened this issue Mar 10, 2022 · 1 comment · Fixed by #46077
Closed

Documentation incorrectly states that C++ new and delete operators are unsupported #43636

waqas-mazhar opened this issue Mar 10, 2022 · 1 comment · Fixed by #46077
Assignees
@stephanosio
Labels
area: C++ area: Documentation bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Milestone
v3.1.0

Comments

@waqas-mazhar
Copy link
Contributor

waqas-mazhar commented Mar 10, 2022 •
edited by stephanosio
Loading

Issue Description
The RTOS documentation, C++ Support for Applications, indicates that C++ Dynamic Objects can't be managed using new/delete operators. However, there is a test case at tests/subsys/cpp/cxx for them. Also, new/delete operators are implemented at subsys/cpp/cpp_new.cpp which eventually resolve into malloc/free respectively both of which are mutex protected. Is there any specific reason due to which they are marked as not supported in the document?

Expected behavior
Document should have more information regarding C++ new/delete support
@waqas-mazhar waqas-mazhar added the bug The issue is a bug, or the PR is fixing a bug label Mar 10, 2022
@stephanosio stephanosio changed the title C++ new and delete operators support in zephyr Documentation incorrectly states that C++ new and delete operators are unsupported Mar 10, 2022
@mbolivar-nordic mbolivar-nordic added the priority: low Low impact/importance bug label Mar 15, 2022
@stephanosio stephanosio added this to the v3.1.0 milestone Mar 29, 2022
@stephanosio stephanosio mentioned this issue May 26, 2022
Closed
@stephanosio
Copy link
Member

Related to #32885

@stephanosio stephanosio mentioned this issue May 27, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stephanosio stephanosio

Labels
area: C++ area: Documentation bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
v3.1.0
Development

Successfully merging a pull request may close this issue.

Toolchains and language support documentation rework stephanosio/zephyr
3 participants
@stephanosio @mbolivar-nordic @waqas-mazhar