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

Fix #735, add comment if null terminated or not. #1168

Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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