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

Remove unused CFE_SB_NO_SUBSCRIBERS #1569

Closed
skliper opened this issue May 24, 2021 · 0 comments · Fixed by #1570
Closed

Remove unused CFE_SB_NO_SUBSCRIBERS #1569

skliper opened this issue May 24, 2021 · 0 comments · Fixed by #1570
Assignees
@skliper
Labels
cFE-SB enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented May 24, 2021

Is your feature request related to a problem? Please describe.
Unused error code. Unsubscribe returns CFE_SUCCESS from an unsubscribe call when there are no subscribers.

Describe the solution you'd like
Remove

Describe alternatives you've considered
None

Additional context
#1452 updated the documentation

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 7.0.0 milestone May 24, 2021
@skliper skliper self-assigned this May 24, 2021
skliper added a commit to skliper/cFE that referenced this issue May 24, 2021
@skliper
Fix nasa#1569, Remove unused CFE_SB_NO_SUBSCRIBERS
eb69c65
@skliper skliper mentioned this issue May 24, 2021
Merged
astrogeco added a commit that referenced this issue May 27, 2021
@astrogeco
Merge pull request #1570 from skliper/fix1452-sb_msg_sbr_comments
786125e 
Fix #1452, #1569, SB/MSG/SBR documentation cleanup and remove unused `CFE_SB_NO_SUBSCRIBERS`
@astrogeco astrogeco mentioned this issue Jun 2, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
cFE-SB enhancement
Projects
None yet
Milestone
Caelum
2 participants
@skliper @astrogeco