Skip to content

Fix bug Operator new[] out of memory #10616

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 1 commit into from
May 23, 2019
Merged

Conversation

ThunderSoft123
Copy link
Contributor

@ThunderSoft123 ThunderSoft123 commented May 20, 2019

Description

Modifying API gpio_init for fix bug when the NC's input value is NC, and changing stack size for fix bug which error is Operator new[] out of memory(error=0x8001011F).

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

Release Notes

@ThunderSoft123 ThunderSoft123 force-pushed the master branch 2 times, most recently from 87edb34 to fff9672 Compare May 20, 2019 08:34
@0xc0170 0xc0170 changed the title Modifiying file gpio_api.c for board TT_M3HQ and TT_M4G9 when pin's value equal NC. TT_M3HQ and TT_M4G9 gpio NC fix May 20, 2019
@kjbracey
Copy link
Contributor

kjbracey commented May 21, 2019

NC fix is already present in #10489. There's another unconnected GPIO fix here though.

@0xc0170
Copy link
Contributor

0xc0170 commented May 21, 2019

NC fix is already present in #10489. There's another unconnected GPIO fix here though.

@ThunderSoft123 Can you align this PR? 10489 will land soon

@ThunderSoft123
Copy link
Contributor Author

@0xc0170 Thank you for support. Do i need to keep the PR to fix bug with Operator new[] out of memory?

@0xc0170
Copy link
Contributor

0xc0170 commented May 21, 2019

@0xc0170 Thank you for support. Do i need to keep the PR to fix bug with Operator new[] out of memory?

Yes, lets update it to only contain this bugfix.

@ThunderSoft123 ThunderSoft123 changed the title TT_M3HQ and TT_M4G9 gpio NC fix Fix bug Operator new[] out of memory May 22, 2019
@0xc0170
Copy link
Contributor

0xc0170 commented May 23, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented May 23, 2019

Test run: SUCCESS

Summary: 11 of 11 test jobs passed
Build number : 1
Build artifacts

@0xc0170 0xc0170 merged commit cfb1079 into ARMmbed:master May 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants