Skip to content

Conversation

@thnkslprpt
Copy link
Contributor

Checklist

Describe the contribution

Some minor points piggy-backing on the PR:

  • typo corrections
  • add cmake_minimum_required
  • change CMake project name to CFS_SAMPLE_APP
  • remove unnecessary single-use variable Crc
  • apply formatting according to cFS bundle .clang-format configuration

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

Expected behavior changes
Avoids risk of size mismatch if decoding/encoding etc.

System(s) tested on
Debian 12 using the current main branch of cFS bundle.

Contributor Info
Avi Weiss   @thnkslprpt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implicit (compiler-added) padding in SAMPLE_APP_Data_t struct

1 participant