Skip to content

adding option to change stack size from compilation #3308

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

Closed
wants to merge 1 commit into from

Conversation

netanelgonen
Copy link
Contributor

Notes:

  • Pull requests will not be accepted until the submitter has agreed to the contributer agreement.
  • This is just a template, so feel free to use/remove the unnecessary things

Description

enable defining the WORDS_STACK_SIZE from compilation
this will create easier way to change the stack size without touching the code

Status

READY

email :netgon01@arm.com

adding option to change stack size from compilation.

this will give easier way to change the stack size without
changing the code.

@theotherjimmy theotherjimmy changed the title user :netanel gonen adding option to change stack size from compilation Nov 21, 2016
@theotherjimmy
Copy link
Contributor

It looks like your intended title did not get picked up correctly by github. If I got the title wrong, please feel free to change it.

@geky
Copy link
Contributor

geky commented Nov 21, 2016

Hi @netanelgonen, thanks for the interest in contributing to mbed-os!

It looks like your commit message may have been messed a bit, would you be able to remove the user/email part of the commit message? These should already be recorded by git.

You should be able to change the commit message with these commands:

git commit --amend
git push origin master --force

email :netgon01@arm.com

adding option to change stack size from compilation.

this will give easyer way to change the stack size without
changing the code.

add the option to set stack size from compilation

adding check if WORDS_STACK_SIZE was not defined before setting it.
so if defined from compilation it will not use the default that set hard coded
it will make controlling the stack size easier.
@c1728p9
Copy link
Contributor

c1728p9 commented Nov 22, 2016

A similar change was made in #2646 and the conclusion was that it masked problems rather than solving them. Because of this the PR was closed. I'm closing this PR for the same reason.

CC: @sg-

@c1728p9 c1728p9 closed this Nov 22, 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