Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

CFE_ES_ERR_SYS_LOG_TRUNCATED Value Isn't Unique #846

Closed
keegan-moore opened this issue Aug 26, 2020 · 0 comments · Fixed by #872 or #876
Closed

CFE_ES_ERR_SYS_LOG_TRUNCATED Value Isn't Unique #846

keegan-moore opened this issue Aug 26, 2020 · 0 comments · Fixed by #872 or #876
Assignees
@skliper
Labels
bug
Milestone
Caelum

Comments

@keegan-moore
Copy link

Describe the bug
CFE_ES_LIB_ALREADY_LOADED and CFE_ES_ERR_SYS_LOG_TRUNCATED are both defined to be ((int32)0x44000028) (in cfe_error.h).

To Reproduce
N/A

Expected behavior
I think CFE_ES_ERR_SYS_LOG_TRUNCATED should instead be set to ((int32)0xC4000029) or some other unique value that follows the status code format.

Code snips
N/A

System observed on:
N/A

Additional context
N/A

Reporter Info
Keegan Moore, NASA/GSFC
@skliper skliper added this to the 7.0.0 milestone Sep 9, 2020
@skliper skliper self-assigned this Sep 9, 2020
skliper added a commit to skliper/cFE that referenced this issue Sep 9, 2020
@skliper
Fix nasa#846, Deconflict CFE_ES_LIB_ALREADY_LOADED and CFE_ES_ERR_SYS…
d01b1bf 
…_LOG_TRUNCATED EIDs
@skliper skliper mentioned this issue Sep 9, 2020
Merged
skliper added a commit to skliper/cFE that referenced this issue Sep 9, 2020
@skliper
Fix nasa#846, Deconflict ES return error codes
7adc055 
CFE_ES_LIB_ALREADY_LOADED and CFE_ES_ERR_SYS_LOG_TRUNCATED
@skliper skliper mentioned this issue Sep 9, 2020
Closed
@jphickey jphickey mentioned this issue Sep 9, 2020
Open
yammajamma added a commit that referenced this issue Sep 10, 2020
@yammajamma
Merge pull request #872 from skliper/fix846-es-error-code-duplicate
d4a4211 
Fix #846, Deconflict CFE_ES_LIB_ALREADY_LOADED and CFE_ES_ERR_SYS_LOG_TRUNCATED
@yammajamma yammajamma mentioned this issue Sep 10, 2020
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@skliper skliper

Labels
bug
Projects
None yet
Milestone
Caelum
2 participants
@skliper @keegan-moore