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

Add app info structure element descriptions back in #2032

Closed
skliper opened this issue Jan 20, 2022 · 0 comments · Fixed by #2033 or #2019
Closed

Add app info structure element descriptions back in #2032

skliper opened this issue Jan 20, 2022 · 0 comments · Fixed by #2033 or #2019
Labels
docs This change only affects documentation. draco-rc1 enhancement
Milestone
Draco

Comments

@skliper
Copy link
Contributor

skliper commented Jan 20, 2022

Is your feature request related to a problem? Please describe.
Element descriptions were dropped in #1029

cFE/modules/core_api/fsw/inc/cfe_es_extern_typedefs.h

Lines 435 to 441 in c161cf0

typedef struct CFE_ES_MemAddOff
{
CFE_ES_MemAddress_t Address;
CFE_ES_MemOffset_t Size;
} CFE_ES_MemAddOff_t;

cFE/modules/core_api/fsw/inc/cfe_es_extern_typedefs.h

Lines 475 to 479 in c161cf0

CFE_ES_MemAddOff_t Code;
CFE_ES_MemAddOff_t Data;
CFE_ES_MemAddOff_t BSS;

Describe the solution you'd like
Add element descriptions

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added enhancement docs This change only affects documentation. labels Jan 20, 2022
@skliper skliper added this to the Draco milestone Jan 20, 2022
skliper added a commit to skliper/cFE that referenced this issue Jan 20, 2022
@skliper
Fix nasa#2032, Add CFE_ES_AppInfo_t element documentation
3cd97e9
@skliper skliper mentioned this issue Jan 20, 2022
Merged
@astrogeco astrogeco mentioned this issue Jan 21, 2022
Merged
astrogeco added a commit that referenced this issue Jan 21, 2022
@astrogeco
Merge pull request #2033 from skliper/fix2032-appinfo_element_doc
47d613c 
Fix #2032, Add CFE_ES_AppInfo_t element documentation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
docs This change only affects documentation. draco-rc1 enhancement
Projects
None yet
Milestone
Draco
1 participant
@skliper