Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent use of TBL vs TABLE #78

Open
astrogeco opened this issue Jun 17, 2020 · 1 comment
Open

Inconsistent use of TBL vs TABLE #78

astrogeco opened this issue Jun 17, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@astrogeco
Copy link
Contributor

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

@astrogeco astrogeco added the bug Something isn't working label Jun 17, 2020
@skliper skliper added enhancement New feature or request and removed bug Something isn't working labels Feb 1, 2021
thnkslprpt added a commit to thnkslprpt/sample_app that referenced this issue Oct 28, 2022
thnkslprpt added a commit to thnkslprpt/sample_app that referenced this issue Oct 28, 2022
@thnkslprpt
Copy link
Contributor

FYI for anyone coming across this - preference is now to use TABLE, not TBL as per the original issue report (in general, to spell out any word <= 6 characters without abbreviation).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants