src/Gtrace: remove unguarded print() calls #483
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There is a use of printf() without #include stdio.h in src/arm/Gtrace.c, which results in a compiler error if clang 16 is used:
src/arm/Gtrace.c:529:7: error: call to undeclared library function 'printf' with type 'int (const char *, ...)'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
Replace the printf("XXX") with a Dprintf, so it doesn't pull stdio in unless in a debug build, and reword the message to be clearer.
Also tehre is another printf("XXX") inside a FreeBSD-specific block in the UNW_ARM_FRAME_SIGRETURN case, replace this with a #error as the code needs to be implemented.
Fixes #482.