Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #2590, Use proper printf format for size_t #2591

Merged
merged 1 commit into from
Aug 22, 2024

Conversation

pepepr08
Copy link
Contributor

@pepepr08 pepepr08 commented Aug 14, 2024

Checklist (Please check before submitting)

Describe the contribution
Fixes #2590 Change the printf format so that the GAISLER RCC compiler doesn't warn about using the incorrect format for printing a size_t variable.

Testing performed
Steps taken to test the contribution:

  1. make distclean && make TARGET=devboard BUILDTYPE=release OMIT_DEPRECATED=true RTEMS_NO_SHELL=false ENABLE_UNIT_TESTS=true -j install

Expected behavior changes
Fully compile

System(s) tested on

  • Hardware: [e.g. PC, SP0, MCP750]
  • OS: [e.g. Ubuntu 18.04, RTEMS 4.11, VxWorks 6.9]
  • Versions: [e.g. cFE 6.6, OSAL 4.2, PSP 1.3 for mcp750, any related apps or tools]

Additional context
Add any other context about the contribution here.

Third party code
If included, identify any third party code and provide text file of license

Contributor Info - All information REQUIRED for consideration of pull request
Full name and company/organization/center of all contributors ("Personal" if individual work)

  • Note CLAs apply to only software contributions.

@pepepr08 pepepr08 added the CCB:Ready Ready for discussion at the Configuration Control Board (CCB) label Aug 15, 2024
@dzbaker dzbaker added CCB:Approved Indicates code review and approval by community CCB and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Aug 15, 2024
dzbaker added a commit to nasa/cFS that referenced this pull request Aug 16, 2024
*Combines:*

cFE equuleus-rc1+dev187
osal equuleus-rc1+dev85
PSP equuleus-rc1+dev48

**Includes:**

*cFE*
- nasa/cFE#2591

*osal*
- nasa/osal#1419

*PSP*
- nasa/PSP#436

Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
Co-authored by: Jose Martinez Pedraza <pepepr08@users.noreply.github.com>
Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com>
dzbaker added a commit to nasa/cFS that referenced this pull request Aug 22, 2024
*Combines:*

cFE equuleus-rc1+dev187
osal equuleus-rc1+dev85

**Includes:**

*cFE*
- nasa/cFE#2591

*osal*
- nasa/osal#1419

Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
Co-authored by: Jose Martinez Pedraza <pepepr08@users.noreply.github.com>
@dzbaker dzbaker merged commit 17a181c into nasa:main Aug 22, 2024
21 checks passed
dzbaker added a commit to nasa/cFS that referenced this pull request Aug 22, 2024
*Combines:*

cFE equuleus-rc1+dev187
osal equuleus-rc1+dev85

**Includes:**

*cFE*
- nasa/cFE#2591

*osal*
- nasa/osal#1419

Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com>
Co-authored by: Jose Martinez Pedraza <pepepr08@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CCB:Approved Indicates code review and approval by community CCB
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compiler warns about printf format of size_t (using Gaisler RCC)
2 participants