apps/sbn/fsw/platform_inc/sbn_types.h and sbn_platform_cfg.h have #include statements that are outside conditional macro style include guard.

Open Beginner friendly
#96 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
1-3 hours
Newbie friendliness
78/100
Issue type
Refactor
Clarity
Clearly specified
Activity status
Quiet
Tech stack
c
Domain
embedded-iot

Research direction

Start with fsw/platform_inc/sbn_types.h and fsw/platform_inc/sbn_platform_cfg.h at the linked line 19 locations, comparing each include guard. Confirm the include placement satisfies the static code analysis rule, then rerun the relevant analysis or project checks; done means both headers pass without the reported guard warning.

Written by the indexing model from the issue text.

Description

Our static code analysis flagged both sbn_types.h and sbn_platform_cfg.h because #include "cfe.h" is not placed within the #ifndef, #define, #endif guard against multiple inclusion.

Dominant language
C
Stars
43
Forks
42
Avg merge
13d 20h
Merged PRs (30d)
2

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from nasa/SBN

All issues in nasa/SBN

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.