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 #217, use size_t in PSP stubs #219

Merged

Conversation

jphickey
Copy link
Contributor

Describe the contribution
Recent API updates require use of the size_t type instead of uint32.

This change is required at the same time as nasa/osal#654 is merged (dependency).

Fixes #217

Testing performed
Build and run all unit tests.

Expected behavior changes
None. No FSW changes, only UT stub update here.

System(s) tested on
Ubuntu 20.04

Additional context
This is only UT stubs. FSW requires API change to use size_t and that will be a separate issue/PR. (see #218)

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

Recent API updates require use of the size_t type
instead of uint32.
@jphickey jphickey added CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) dependency labels Nov 17, 2020
@astrogeco
Copy link
Contributor

CCB 2020-11-18 APPROVED

@astrogeco astrogeco added IC-20201124 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Nov 23, 2020
@astrogeco astrogeco changed the base branch from main to integration-candidate November 23, 2020 03:44
@astrogeco astrogeco merged commit dd9fd53 into nasa:integration-candidate Nov 23, 2020
astrogeco added a commit to astrogeco/cFS that referenced this pull request Nov 23, 2020
astrogeco added a commit to astrogeco/cFS that referenced this pull request Dec 2, 2020
@jphickey jphickey deleted the fix-217-stubs-use-size_t branch January 6, 2021 19:34
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.

Update unit test stubs to use size_t
2 participants