Skip to content

debug.h: check stacksize fixed#2747

Merged
miri64 merged 1 commit intoRIOT-OS:masterfrom
brummer-simon:fix-debug_makro
Mar 31, 2015
Merged

debug.h: check stacksize fixed#2747
miri64 merged 1 commit intoRIOT-OS:masterfrom
brummer-simon:fix-debug_makro

Conversation

@brummer-simon
Copy link
Copy Markdown
Member

Changed Stacksize check im DEBUG-Makro from > KERNEL_CONF_STACKSIZE_PRINTF to >= KERNEL_CONF_STACKSIZE_PRINTF.

This makes usage of the DEBUG Makro possible if the calling thread has a stacksize thats intended for Printf usage.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 31, 2015

ACK when travis is happy

@miri64 miri64 self-assigned this Mar 31, 2015
@miri64 miri64 added the Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) label Mar 31, 2015
@OlegHahm OlegHahm force-pushed the master branch 2 times, most recently from 9f184dd to 45554bf Compare March 31, 2015 13:01
@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 31, 2015

And go

miri64 added a commit that referenced this pull request Mar 31, 2015
@miri64 miri64 merged commit 2fe68d5 into RIOT-OS:master Mar 31, 2015
@brummer-simon brummer-simon deleted the fix-debug_makro branch March 31, 2015 16:37
@kaspar030
Copy link
Copy Markdown
Contributor

huh, didn't we have that before? #2183 #1244
This was a one-acked core change... edit ... that was NACKed two times before. Martine, can you revert?

@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 31, 2015

Done.

@miri64
Copy link
Copy Markdown
Member

miri64 commented Mar 31, 2015

(also see #2725)

@PeterKietzmann
Copy link
Copy Markdown
Member

I think we can close this PR or #2725, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants