Skip to content

Commit ab21c87

Browse files
authored
Merge pull request nasa#1168 from zanzaben/fix735_document_null_terminators
Fix nasa#735, add comment if null terminated or not.
2 parents 9fd5d76 + 81e907f commit ab21c87

File tree

1 file changed

+11
-1
lines changed

1 file changed

+11
-1
lines changed

cmake/sample_defs/sample_mission_cfg.h

+11-1
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,6 @@
4747
** messages sent. If the pkt length field indicates the message is larger
4848
** than this define, SB sends an event and rejects the send.
4949
**
50-
**
5150
** \par Limits
5251
** This parameter has a lower limit of 6 (CCSDS primary header size). There
5352
** are no restrictions on the upper limit however, the maximum message size is
@@ -247,6 +246,7 @@
247246
** portion of a Full CDS Name of the following form:
248247
** "ApplicationName.CDSName"
249248
**
249+
** This length does not need to include an extra character for NULL termination.
250250
**
251251
** \par Limits
252252
** This value should be kept as a multiple of 4, to maintain alignment of
@@ -264,6 +264,8 @@
264264
** Indicates the maximum length (in characters) of the formatted text
265265
** string portion of an event message
266266
**
267+
** This length does not need to include an extra character for NULL termination.
268+
**
267269
** \par Limits
268270
** Not Applicable
269271
*/
@@ -299,6 +301,8 @@
299301
** ('TblName') portion of a Full Table Name of the following
300302
** form: "ApplicationName.TblName"
301303
**
304+
** This length does not need to include an extra character for NULL termination.
305+
**
302306
** \par Limits
303307
** This value should be kept as a multiple of 4, to maintain alignment of
304308
** any possible neighboring fields without implicit padding.
@@ -531,6 +535,8 @@
531535
**
532536
** This affects only the layout of command/telemetry messages and table definitions;
533537
** internal allocation may use the platform-specific OS_MAX_PATH_LEN value.
538+
**
539+
** This length must include an extra character for NULL termination.
534540
**
535541
** \par Limits
536542
** All CPUs within the same SB domain (mission) and ground tools must share the
@@ -557,6 +563,8 @@
557563
** This affects only the layout of command/telemetry messages and table definitions;
558564
** internal allocation may use the platform-specific OS_MAX_FILE_LEN value.
559565
**
566+
** This length must include an extra character for NULL termination.
567+
**
560568
** \par Limits
561569
** All CPUs within the same SB domain (mission) and ground tools must share the
562570
** same definition.
@@ -582,6 +590,8 @@
582590
** This affects only the layout of command/telemetry messages and table definitions;
583591
** internal allocation may use the platform-specific OS_MAX_API_LEN value.
584592
**
593+
** This length must include an extra character for NULL termination.
594+
**
585595
** \par Limits
586596
** All CPUs within the same SB domain (mission) must share the same definition
587597
** Note this affects the size of messages, so it must not cause any message

0 commit comments

Comments
 (0)