Skip to content

Commit

Permalink
Fix #735, comment if null terminated or not.
Browse files Browse the repository at this point in the history
  • Loading branch information
zanzaben committed Feb 16, 2021
1 parent fcaa1d0 commit ec65900
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions cmake/sample_defs/sample_mission_cfg.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@
** messages sent. If the pkt length field indicates the message is larger
** than this define, SB sends an event and rejects the send.
**
** This length does not need to include an extra character for NULL termination.
**
** \par Limits
** This parameter has a lower limit of 6 (CCSDS primary header size). There
Expand Down Expand Up @@ -247,6 +248,7 @@
** portion of a Full CDS Name of the following form:
** "ApplicationName.CDSName"
**
** This length does not need to include an extra character for NULL termination.
**
** \par Limits
** This value should be kept as a multiple of 4, to maintain alignment of
Expand All @@ -264,6 +266,8 @@
** Indicates the maximum length (in characters) of the formatted text
** string portion of an event message
**
** This length does not need to include an extra character for NULL termination.
**
** \par Limits
** Not Applicable
*/
Expand Down Expand Up @@ -299,6 +303,8 @@
** ('TblName') portion of a Full Table Name of the following
** form: "ApplicationName.TblName"
**
** This length does not need to include an extra character for NULL termination.
**
** \par Limits
** This value should be kept as a multiple of 4, to maintain alignment of
** any possible neighboring fields without implicit padding.
Expand Down Expand Up @@ -531,6 +537,8 @@
**
** This affects only the layout of command/telemetry messages and table definitions;
** internal allocation may use the platform-specific OS_MAX_PATH_LEN value.
**
** This length must include an extra character for NULL termination.
**
** \par Limits
** All CPUs within the same SB domain (mission) and ground tools must share the
Expand All @@ -557,6 +565,8 @@
** This affects only the layout of command/telemetry messages and table definitions;
** internal allocation may use the platform-specific OS_MAX_FILE_LEN value.
**
** This length must include an extra character for NULL termination.
**
** \par Limits
** All CPUs within the same SB domain (mission) and ground tools must share the
** same definition.
Expand All @@ -582,6 +592,8 @@
** This affects only the layout of command/telemetry messages and table definitions;
** internal allocation may use the platform-specific OS_MAX_API_LEN value.
**
** This length must include an extra character for NULL termination.
**
** \par Limits
** All CPUs within the same SB domain (mission) must share the same definition
** Note this affects the size of messages, so it must not cause any message
Expand Down

0 comments on commit ec65900

Please sign in to comment.