nasa / sample_app

The Core Flight System (cFS) Sample App (sample_app)
Apache License 2.0
49 stars 43 forks source link

Fix #78, Align use of Table/Tbl mnemonics #191

Closed thnkslprpt closed 2 years ago

thnkslprpt commented 2 years ago

Checklist

Describe the contribution Fixes #78 Aligns use of Table/Tbl mnemonics in sample_app. The macros and function parameters that used 'table' have been converted to 'tbl'. I left the table struct (SAMPLE_APP_Table_t) as is for now, because this is the more common convention in cFS.

Testing performed GitHub CI actions (incl. Build + Run, Unit Tests etc.) all passing successfully.

Expected behavior changes No impact on behavior.

Contributor Info Avi Weiss @thnkslprpt

dzbaker commented 2 years ago

CCB 3 November 2022: Preference would be to spell out "table" in all cases rather than spell it "tbl" (according to coding standards). Closing for now.

thnkslprpt commented 2 years ago

CCB 3 November 2022: Preference would be to spell out "table" in all cases rather than spell it "tbl" (according to coding standards). Closing for now.

Just to clarify @dzbaker - was the preference now for the all use of 'tbl' to be converted to 'table' (and 'TBL' to 'TABLE') in sample_app, or to close that issue without any action?

dzbaker commented 2 years ago

CCB 3 November 2022: Preference would be to spell out "table" in all cases rather than spell it "tbl" (according to coding standards). Closing for now.

Just to clarify @dzbaker - was the preference now for the all use of 'tbl' to be converted to 'table' (and 'TBL' to 'TABLE') in sample_app, or to close that issue without any action?

Hi @thnkslprpt - Yes, the preferable format would ideally be 'TABLE' (typically, we'll spell out anything that is 6 or fewer characters). We discussed this at yesterday's CCB. Ideally, we would like to make them all consistent, but there's the risk that we'll miss something that will inadvertently result in something breaking somewhere, so we'd like to tackle formatting issues like these when we have more time/resources to fully test the changes.

thnkslprpt commented 2 years ago

CCB 3 November 2022: Preference would be to spell out "table" in all cases rather than spell it "tbl" (according to coding standards). Closing for now.

Just to clarify @dzbaker - was the preference now for the all use of 'tbl' to be converted to 'table' (and 'TBL' to 'TABLE') in sample_app, or to close that issue without any action?

Hi @thnkslprpt - Yes, the preferable format would ideally be 'TABLE' (typically, we'll spell out anything that is 6 or fewer characters). We discussed this at yesterday's CCB. Ideally, we would like to make them all consistent, but there's the risk that we'll miss something that will inadvertently result in something breaking somewhere, so we'd like to tackle formatting issues like these when we have more time/resources to fully test the changes.

Cool no worries. Thanks for the clarification.