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.

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

printf format issue on RTEMS #1522

Closed
jphickey opened this issue May 13, 2021 · 0 comments · Fixed by #1524 or #1568
Closed

printf format issue on RTEMS #1522

jphickey opened this issue May 13, 2021 · 0 comments · Fixed by #1524 or #1568
Milestone

Comments

@jphickey
Copy link
Contributor

Describe the bug
When compiling the test app under RTEMS, a build failure occurs:

/home/joe/code/cfecfs/github/cfe/modules/cfe_testcase/src/es_task_test.c: In function 'TestCreateChild':
/home/joe/code/cfecfs/github/cfe/modules/cfe_testcase/src/es_task_test.c:89:5: error: format '%d' expects argument of type 'int', but argument 7 has type 'uint32' [-Werror=format=]
     UtAssert_True(countCopy == count || countCopy == count + 1, "countCopy (%d) == count (%d)", countCopy, count);
     ^

To Reproduce
Build system for RTEMS 4.11 with unit tests enabled, and es_task_test.c will fail to build.

Expected behavior
Build should succeed.

System observed on:
RTEMS 4.11

Reporter Info
Joseph Hickey, Vantage Systems, Inc.

jphickey added a commit that referenced this issue May 13, 2021
Use (int) with %d conversions for portability
jphickey added a commit to jphickey/cFE that referenced this issue May 13, 2021
Use (int) with %d conversions for portability
jphickey added a commit to jphickey/cFE that referenced this issue May 14, 2021
Use (int) with %d conversions for portability
astrogeco added a commit that referenced this issue May 24, 2021
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants