Consider replacing switch statement with jump table in CS_BackgroundCheckCmd
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- C
- Stars
- 36
- Forks
- 41
- Avg merge
- 11d 19h
- Merged PRs (30d)
- 2
Description
Finding from code review
Imported from GSFCCFS-1324
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate CS_BackgroundCheckCmd in the CFS Checksum application and read the surrounding implementation and any related tests or build guidance. Determine whether the switch should be replaced and what behavior must remain unchanged; done means an agreed, tested implementation, but the issue names no file or test.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- embedded-iot
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100