Skip to content

Fix #988, add casts on printf calls#990

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
jphickey:fix-988-printf-warnings
Nov 3, 2020
Merged

Fix #988, add casts on printf calls#990
astrogeco merged 1 commit intonasa:integration-candidatefrom
jphickey:fix-988-printf-warnings

Conversation

@jphickey
Copy link
Contributor

@jphickey jphickey commented Nov 2, 2020

Describe the contribution
Cast fixed width types to the type used in the printf call.

Fixes #988

Testing performed
Build and sanity test on RTEMS target

Expected behavior changes
No warnings related to printf

System(s) tested on
Ubuntu 20.04 host for i686-rtems4.11 target

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

Cast fixed width types to the type used in the printf call
@jphickey jphickey added CCB:FastTrack CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Nov 2, 2020
@jphickey
Copy link
Contributor Author

jphickey commented Nov 2, 2020

Suggest fast track on this one - the issue causes the current "main" to not build - and the fix is simple enough.

@jphickey jphickey requested a review from skliper November 2, 2020 14:55
@astrogeco astrogeco changed the base branch from main to integration-candidate November 3, 2020 14:44
@astrogeco astrogeco merged commit df2de6c into nasa:integration-candidate Nov 3, 2020
astrogeco added a commit to nasa/cFS that referenced this pull request Nov 3, 2020
@astrogeco astrogeco added CCB-20201104 CCB:FastTrack and removed CCB:FastTrack CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Nov 4, 2020
@jphickey jphickey deleted the fix-988-printf-warnings branch December 3, 2020 17:53
@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

Development

Successfully merging this pull request may close these issues.

New printf warnings in RTEMS build

3 participants