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

sch_lab Integration candidate: Caelum-rc4+dev64 #154

Merged
merged 9 commits into from
Nov 14, 2023
4 changes: 2 additions & 2 deletions .github/workflows/codeql-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,12 +3,12 @@ name: "CodeQL Analysis"
on:
push:
pull_request:

jobs:
codeql:
name: Codeql
uses: nasa/cFS/.github/workflows/codeql-reusable.yml@main
with:
with:
component-path: apps/sch_lab
prep: 'make prep; make -C build/tools/elf2cfetbl'
make: 'make -C build/native/default_cpu1/apps/sch_lab'
1 change: 0 additions & 1 deletion .github/workflows/format-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,3 @@ jobs:
format-check:
name: Run format check
uses: nasa/cFS/.github/workflows/format-check.yml@main

1 change: 0 additions & 1 deletion .github/workflows/static-analysis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,3 @@ jobs:
static-analysis:
name: Run cppcheck
uses: nasa/cFS/.github/workflows/static-analysis.yml@main

7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
# Changelog

## Development Build: v2.5.0-rc4+dev71
- Standardize naming of SCH_LAB functions/macros
- Call CFE_ES_ExitApp with RunStatus rather than internal status variable
- Convert int32 return codes and variables to CFE_Status_t
- Remove component-specific cFE header #includes
- See <https://github.com/nasa/sch_lab/pull/129>, <https://github.com/nasa/sch_lab/pull/149>, <https://github.com/nasa/sch_lab/pull/142>, and <https://github.com/nasa/sch_lab/pull/134>

## Development Build: v2.5.0-rc4+dev61
- implement header file convention for SCH_LAB
- See <https://github.com/nasa/sch_lab/pull/151>
Expand Down
2 changes: 1 addition & 1 deletion config/default_sch_lab_perfids.h
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,6 @@
#ifndef SCH_LAB_PERFIDS_H
#define SCH_LAB_PERFIDS_H

#define SCH_MAIN_TASK_PERF_ID 36
#define SCH_LAB_MAIN_TASK_PERF_ID 36

#endif
2 changes: 1 addition & 1 deletion config/default_sch_lab_tbl.h
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,6 @@
* There is no extra encapsulation here, this header only
* defines the default file name to use for the SCH table
*/
#define SCH_TBL_DEFAULT_FILE "/cf/sch_lab_table.tbl"
#define SCH_LAB_TBL_DEFAULT_FILE "/cf/sch_lab_table.tbl"

#endif
18 changes: 9 additions & 9 deletions fsw/src/sch_lab_app.c
Original file line number Diff line number Diff line change
Expand Up @@ -63,12 +63,12 @@
/*
** Local Function Prototypes
*/
int32 SCH_LAB_AppInit(void);
CFE_Status_t SCH_LAB_AppInit(void);

/*
** AppMain
*/
void SCH_Lab_AppMain(void)
void SCH_LAB_AppMain(void)

Check notice

Code scanning / CodeQL-coding-standard

Long function without assertion Note

All functions of more than 10 lines should have at least one assertion.

Check notice

Code scanning / CodeQL-coding-standard

Function too long Note

SCH_LAB_AppMain has too many lines (69, while 60 are allowed).
{
int i;
uint32 SCH_OneHzPktsRcvd = 0;
Expand All @@ -78,7 +78,7 @@
SCH_LAB_StateEntry_t *LocalStateEntry;
CFE_SB_Buffer_t * SBBufPtr;

CFE_ES_PerfLogEntry(SCH_MAIN_TASK_PERF_ID);
CFE_ES_PerfLogEntry(SCH_LAB_MAIN_TASK_PERF_ID);

Status = SCH_LAB_AppInit();
if (Status != CFE_SUCCESS)
Expand All @@ -90,7 +90,7 @@
/* Loop Forever */
while (CFE_ES_RunLoop(&RunStatus) == true)
{
CFE_ES_PerfLogExit(SCH_MAIN_TASK_PERF_ID);
CFE_ES_PerfLogExit(SCH_LAB_MAIN_TASK_PERF_ID);

/* Pend on timing sem */
OsStatus = OS_CountSemTake(SCH_LAB_Global.TimingSem);
Expand All @@ -104,7 +104,7 @@
Status = CFE_STATUS_EXTERNAL_RESOURCE_FAIL;
}

CFE_ES_PerfLogEntry(SCH_MAIN_TASK_PERF_ID);
CFE_ES_PerfLogEntry(SCH_LAB_MAIN_TASK_PERF_ID);

if (Status == CFE_SUCCESS)
{
Expand Down Expand Up @@ -135,7 +135,7 @@

} /* end while */

CFE_ES_ExitApp(Status);
CFE_ES_ExitApp(RunStatus);
}

void SCH_LAB_LocalTimerCallback(osal_id_t object_id, void *arg)
Expand All @@ -148,10 +148,10 @@
/* Initialization */
/* */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
int32 SCH_LAB_AppInit(void)
CFE_Status_t SCH_LAB_AppInit(void)

Check notice

Code scanning / CodeQL-coding-standard

Function too long Note

SCH_LAB_AppInit has too many lines (154, while 60 are allowed).

Check notice

Code scanning / CodeQL-coding-standard

Long function without assertion Note

All functions of more than 10 lines should have at least one assertion.
{
int i, x;
int32 Status;
CFE_Status_t Status;
int32 OsStatus;
uint32 TimerPeriod;
osal_id_t TimeBaseId = OS_OBJECT_ID_UNDEFINED;
Expand Down Expand Up @@ -202,7 +202,7 @@
/*
** Loading Table
*/
Status = CFE_TBL_Load(SCH_LAB_Global.TblHandle, CFE_TBL_SRC_FILE, SCH_TBL_DEFAULT_FILE);
Status = CFE_TBL_Load(SCH_LAB_Global.TblHandle, CFE_TBL_SRC_FILE, SCH_LAB_TBL_DEFAULT_FILE);
if (Status != CFE_SUCCESS)
{
CFE_ES_WriteToSysLog("SCH_LAB: Error Loading Table ScheduleTable, RC = 0x%08lX\n", (unsigned long)Status);
Expand Down
2 changes: 1 addition & 1 deletion fsw/src/sch_lab_version.h
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
*/

/* Development Build Macro Definitions */
#define SCH_LAB_BUILD_NUMBER 61 /*!< Development Build: Number of commits since baseline */
#define SCH_LAB_BUILD_NUMBER 71 /*!< Development Build: Number of commits since baseline */
#define SCH_LAB_BUILD_BASELINE \
"v2.5.0-rc4" /*!< Development Build: git tag that is the base for the current development */

Expand Down
Loading