Skip to content

Commit c86efce

Browse files
Apply suggestions from code review
Co-authored-by: Miguel Company <miguelcompany@eprosima.com>
1 parent 7c28491 commit c86efce

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

srcCxx/shape_main.cxx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -898,10 +898,10 @@ class ShapeOptions {
898898
parse_ok = false;
899899
}
900900
// the spec mentions that the fragment size must satisfy:
901-
// fragment size <= 65536 bytes.
902-
if (converted_param > 65536) {
901+
// fragment size <= 65535 bytes.
902+
if (converted_param > 65535) {
903903
logger.log_message("incorrect value for datafrag-size, "
904-
"it must be <= 65536 bytes"
904+
"it must be <= 65535 bytes"
905905
+ std::to_string(converted_param),
906906
Verbosity::ERROR);
907907
parse_ok = false;
@@ -952,7 +952,7 @@ class ShapeOptions {
952952
+ (unregister ? "Unregister" : (dispose ? "Dispose" : "not specified")) +
953953
"\n Periodic Announcement Period = "
954954
+ std::to_string(periodic_announcement_period_us / 1000) + "ms" +
955-
"\n Data Fragmentation Size = " + std::to_string(datafrag_size) + "ms",
955+
"\n Data Fragmentation Size = " + std::to_string(datafrag_size) + " bytes",
956956
Verbosity::DEBUG);
957957
if (topic_name != NULL){
958958
logger.log_message(" Topic = " + std::string(topic_name),

0 commit comments

Comments
 (0)