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 #718, deprecate OS_fsBlocksFree and OS_fsBytesFree #759

Conversation

jphickey
Copy link
Contributor

@jphickey jphickey commented Jan 14, 2021

Describe the contribution
Deprecates the OS_fsBlocksFree() and OS_fsBytesFree() functions because the return value is inadequate for larger file systems.

Users should call OS_FileSysStatVolume() and read the respective output members of the structure as indicated.

Fixes #718

Testing performed
Build CFE and unit test with both OMIT_DEPRECATED as true and false

Expected behavior changes
Any calls to these functions will become an error if building with OMIT_DEPRECATED=true

System(s) tested on
Ubuntu 20.04

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

Users should call OS_FileSysStatVolume() and read the respective
output members of the structure as indicated.
@skliper skliper added this to the 6.0.0 milestone Jan 15, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate January 21, 2021 14:38
@astrogeco astrogeco merged commit ff42bfd into nasa:integration-candidate Jan 21, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jan 21, 2021
@jphickey jphickey deleted the fix-718-deprecate-bytesblocksfree branch January 27, 2021 14:09
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
jphickey pushed a commit to jphickey/osal that referenced this pull request Aug 10, 2022
…_deprecated

fix nasa#759 - deprecates GetLastSenderId()
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.

Deprecate OS_fsBlocksFree and OS_fsBytesFree
3 participants