Replace use of `goto`, coding standard violation

skliper commented 2 years ago

Checklist (Please check before submitting)

[x] I reviewed the Contributing Guide.
[x] I reviewed the CF README.md file to see if the feature is in the major future work.
[x] I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

Is your feature request related to a problem? Please describe. goto violates power of ten and JPL coding standards, typically avoided in cFS code.

Describe the solution you'd like Refactor

Describe alternatives you've considered None

Additional context Static analysis warning that showed up in #218 but wasn't resolved as part of that issue fix

Requester Info Jacob Hageman - NASA/GSFC

astrogeco commented 2 years ago

@skliper how do you feel about adding this to Draco?

skliper commented 2 years ago

@astrogeco It's non-trivial, but if we've got the resources it needs to get done.

skliper commented 2 years ago

Adding to Draco for tracking as @havencarlson has it close to resolved and good to close this violation if there's time this round. Can always reassess if we run out of time.

nasa / CF

Replace use of `goto`, coding standard violation #225