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

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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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