nasa / nasa/sample_app

Inconsistent use of TBL vs TABLE

Open
#78 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C
Stars
67
Forks
60
PR merge metrics
No merged PRs in 30d

Description

Describe the bug
It seems we use TBL and TABLE interchangeably. For example

  status = CFE_TBL_Register(&SAMPLE_AppData.TblHandles[0],
                              "SampleAppTable",
                              sizeof(SAMPLE_APP_Table_t),
                              CFE_TBL_OPT_DEFAULT,
                              SAMPLE_TblValidationFunc);

has both uses.

To Reproduce
Search the sample_app directory for uses of TBL and TABLE both upper and lower case combinations exist.

Expected behavior
A single mnemonic for tables. TBL seems to be the default used in cFS.

Reporter Info
Gerardo E. Cruz-Ortiz, NASA-GSFC

Contributor guide

Open the contributing guide

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.

Research direction

Search the sample_app directory for all upper- and lowercase combinations of TBL and TABLE, starting with the examples shown in the issue. Standardize the table mnemonic to TBL throughout the affected names and references, then verify that no inconsistent TABLE variants remain.

Written by the indexing model from the issue text.

Assessment

Tech stack
c
Domain
embedded-iot
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.