CF Code Style and Coding standards compliance about cf

[jphickey]

Imported comment by internal user on Tue Nov 16 16:43:32 2021

CF also has a number of functions that have a trailing underscore (e.g. CF_CFDP_PlaybackDir_) for no apparent reason. This should be corrected and the underscore removed

[jphickey]

Imported comment by internal user on Wed Nov 17 09:16:55 2021

In addition to above, there are also function names that differ only in the presence (or not) of a trailing underscore. For example:

CF_CFDP_FindTransactionBySequenceNumber{{ }}

CF_CFDP_FindTransactionBySequenceNumber_

The former is a global (app-wide) call, where the latter is a local helper function to implement that call. However I understand coding standards to require a greater differentiation in the function names than just a trailing underscore.

[jphickey]

Imported comment by internal user on Wed Nov 17 11:22:05 2021

Another specific area of concern/issue here is with regard to the "clist_node" typedef. It is defined as follows:
struct clist_node_t;
typedef struct clist_node_t

{
struct clist_node_t *next;
struct clist_node_t *prev;
} clist_node_t;
typedef struct clist_node_t *clist_node;

Typically in the CFS patterns/style the bare struct would be named "clist_node", but in this case that is a pointer typedef. In general the CFE code avoids pointer typedefs, they can be confusing - just including the "*" generally yields clearer code because its obvious the value is a pointer, and doesn't cause strangeness WRT the "const" keyword.

[jphickey]

Since scrubbing for adherence to the CFS conventions for naming and style is likely to be a large changeset, this should be split into several stages:

• Initial Pass thru clang-format was already done in #16 (merged)
• Each header + source file should be individually scrubbed for naming convention/scope issues.

Naming convention issues are mostly just search and replace, so little chance for unintended consequences if done properly - but they do affect many files, so unlikely to be able to merge other changes across it.

[jphickey]

The linked PR fixes the names of identifiers in headers, but still does not address the other issues, specifically declaring variables in the middle of functions, and internal helper functions which are not declared in headers (specifically the ones that end in underscores).

I'd like to close this issue with PR #98 and I will submit additional issues for the remaining items.

[jphickey]

Note - issues #109 and #110 are for the items noted above but not fixed in PR #98