Skip to content

Conversation

@thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

Testing performed
GitHub CI actions all passing successfully (incl. Build + Run, Unit/Functional Tests etc.).

Expected behavior changes
Simplifies CS_ComputeApp.

System(s) tested on
Debian 12 using the current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

Copy link
Contributor

@chillfig chillfig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Function behavior is retained and superfluous variable removed. Good change.

@dzbaker dzbaker merged commit fff5114 into nasa:main Apr 3, 2025
16 checks passed
@thnkslprpt thnkslprpt deleted the fix-103-remove-redundant-Result-variable-in-CS_ComputeApp branch April 4, 2025 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Result variable in CS_ComputeApp() could be removed

3 participants