Add TOPIC_DATA to ddsi_default_qos_topic

No externally visible effect for applications, but not setting is wrong and affects the C++ QoS provider implementation because it directly looks at the default QoS objects. Signed-off-by: Erik Boasson <[email protected]>
eclipse-cyclonedds · Apr 10, 2024 · 6498bf9 · 6498bf9
1 parent 826a070
commit 6498bf9
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/core/ddsi/src/ddsi_plist.c b/src/core/ddsi/src/ddsi_plist.c
@@ -3662,7 +3662,7 @@ const dds_qos_t ddsi_default_qos_writer = {
 };
 
 const dds_qos_t ddsi_default_qos_topic = {
-  .present = DDSI_QP_PRESENTATION | DDSI_QP_DURABILITY | DDSI_QP_DEADLINE | DDSI_QP_LATENCY_BUDGET | DDSI_QP_LIVELINESS | DDSI_QP_DESTINATION_ORDER | DDSI_QP_HISTORY | DDSI_QP_RESOURCE_LIMITS | DDSI_QP_TRANSPORT_PRIORITY | DDSI_QP_OWNERSHIP | DDSI_QP_CYCLONE_IGNORELOCAL | DDSI_QP_DURABILITY_SERVICE | DDSI_QP_RELIABILITY | DDSI_QP_LIFESPAN | DDSI_QP_DATA_REPRESENTATION,
+  .present = DDSI_QP_PRESENTATION | DDSI_QP_DURABILITY | DDSI_QP_DEADLINE | DDSI_QP_LATENCY_BUDGET | DDSI_QP_LIVELINESS | DDSI_QP_DESTINATION_ORDER | DDSI_QP_HISTORY | DDSI_QP_RESOURCE_LIMITS | DDSI_QP_TOPIC_DATA | DDSI_QP_TRANSPORT_PRIORITY | DDSI_QP_OWNERSHIP | DDSI_QP_CYCLONE_IGNORELOCAL | DDSI_QP_DURABILITY_SERVICE | DDSI_QP_RELIABILITY | DDSI_QP_LIFESPAN | DDSI_QP_DATA_REPRESENTATION,
   .aliased = DDSI_QP_DATA_REPRESENTATION,
   .presentation.access_scope = DDS_PRESENTATION_INSTANCE,
   .presentation.coherent_access = 0,
@@ -3678,6 +3678,8 @@ const dds_qos_t ddsi_default_qos_topic = {
   .resource_limits.max_samples = DDS_LENGTH_UNLIMITED,
   .resource_limits.max_instances = DDS_LENGTH_UNLIMITED,
   .resource_limits.max_samples_per_instance = DDS_LENGTH_UNLIMITED,
+  .topic_data.length = 0,
+  .topic_data.value = NULL,
   .transport_priority.value = 0,
   .ownership.kind = DDS_OWNERSHIP_SHARED,
   .ignorelocal.value = DDS_IGNORELOCAL_NONE,