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.

Sign up for GitHub

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

Jump to bottom

CFE_FS_ReadHeader & WriteHeader api headers state wrong returns #1299

Closed
zanzaben opened this issue Apr 12, 2021 · 0 comments · Fixed by #1598 or #1619
Closed

CFE_FS_ReadHeader & WriteHeader api headers state wrong returns #1299

zanzaben opened this issue Apr 12, 2021 · 0 comments · Fixed by #1598 or #1619
Assignees
@skliper
Labels
docs This change only affects documentation. enhancement
Milestone
Caelum

Comments

@zanzaben
Copy link
Contributor

Describe the bug
In the header api's for ReadHeader and WriteHeader they both say that they return the execution status, however the implementation has them return the size of the file they read/wrote if successful and an error code if not.

Reporter Info
Alex Campbell GSFC
@zanzaben zanzaben added the docs This change only affects documentation. label Apr 12, 2021
@skliper skliper added this to the 7.0.0 milestone Apr 12, 2021
@skliper skliper mentioned this issue Jun 3, 2021
Merged
@skliper skliper added enhancement and removed docs This change only affects documentation. labels Jun 3, 2021
@skliper skliper self-assigned this Jun 3, 2021
@skliper skliper added the docs This change only affects documentation. label Jun 3, 2021
skliper added a commit to skliper/cFE that referenced this issue Jun 3, 2021
@skliper
Fix nasa#1299, Updated FS Read/WriteHeader API return documentation
4064777
@skliper skliper mentioned this issue Jun 3, 2021
Merged
astrogeco added a commit that referenced this issue Jun 10, 2021
@astrogeco
Merge pull request #1598 from skliper/fix1299-fs_rtn_codes
eda650f 
Fix #1299, Updated FS Read/WriteHeader API return documentation
@astrogeco astrogeco mentioned this issue Jun 15, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
docs This change only affects documentation. enhancement
Projects
None yet
Milestone
Caelum
2 participants
@zanzaben @skliper