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

Create structure to associated CFE_ES_MemAddress_t and CFE_ES_MemOffset_t #1300

Closed
skliper opened this issue Apr 12, 2021 · 1 comment · Fixed by #1911 or #2001
Closed

Create structure to associated CFE_ES_MemAddress_t and CFE_ES_MemOffset_t #1300

skliper opened this issue Apr 12, 2021 · 1 comment · Fixed by #1911 or #2001
Assignees
@astrogeco
Labels
CFS-40 draco-rc1 wontfix
Milestone
Draco

Comments

@skliper
Copy link
Contributor

skliper commented Apr 12, 2021

Is your feature request related to a problem? Please describe.
See #1283 (comment)

CFE_ES_MemAddres_t and CFE_ES_MemOffset_t are often used in pairs.

Describe the solution you'd like
The two could be combined into (for example) CFE_ES_MemAddOff_t

Describe alternatives you've considered
None

Additional context
From code review #1283

Requester Info
Jacob Hageman NASA/GSFC, submitted by @stashakk
@skliper skliper mentioned this issue Apr 12, 2021
Closed
@skliper skliper added the CFS-40 label Apr 13, 2021
@paulober paulober mentioned this issue Aug 30, 2021
Merged
@astrogeco astrogeco added this to the cFS-Draco milestone Sep 7, 2021
paulober pushed a commit to paulober/cFE that referenced this issue Oct 21, 2021
Fix nasa#1300, Add CFE_ES_MemAddOff structure
e91275e
@astrogeco astrogeco self-assigned this Oct 28, 2021
astrogeco added a commit to astrogeco/cFE that referenced this issue Oct 29, 2021
@astrogeco
Merge pull request nasa#1911 from pavll/fix-1300-es-memaddoff-struct
d7abb46 
Fix nasa#1300, CFE_ES_MemAddOff structure to associated CFE_ES_MemAddress_t
and CFE_ES_MemOffset_t
@astrogeco astrogeco mentioned this issue Nov 17, 2021
Merged
This was referenced Jan 25, 2022
Closed
Merged
@astrogeco astrogeco reopened this Jan 27, 2022
@astrogeco
Copy link
Contributor

CCB:2022-01-27,

  • Will wait until next major release to implement

@skliper skliper added wontfix and removed enhancement good first issue Good for newcomers labels Feb 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@astrogeco astrogeco

Labels
CFS-40 draco-rc1 wontfix
Projects
None yet
Milestone
Draco
2 participants
@skliper @astrogeco