Skip to content

Commit

Permalink
Fix nasa#735, comment if null terminated or not.
Browse files Browse the repository at this point in the history
  • Loading branch information
zanzaben committed Feb 17, 2021
1 parent fcaa1d0 commit 81e907f
Showing 1 changed file with 11 additions and 1 deletion.
12 changes: 11 additions & 1 deletion cmake/sample_defs/sample_mission_cfg.h
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,6 @@
** messages sent. If the pkt length field indicates the message is larger
** than this define, SB sends an event and rejects the send.
**
**
** \par Limits
** This parameter has a lower limit of 6 (CCSDS primary header size). There
** are no restrictions on the upper limit however, the maximum message size is
Expand Down Expand Up @@ -247,6 +246,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 +264,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 +301,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 +535,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 +563,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 +590,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 81e907f

Please sign in to comment.