to support reset content filtered topic with empty string.

remove constness for rmw_subscription because is_cft_supported might be updated Signed-off-by: Chen Lihui <[email protected]>
ros2 · Mar 12, 2021 · ee73588 · ee73588
1 parent 355ed92
commit ee73588
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/rmw_connext_cpp/src/rmw_subscription.cpp b/rmw_connext_cpp/src/rmw_subscription.cpp
@@ -269,6 +269,7 @@ rmw_create_subscription(
 #endif
 
   subscription->can_loan_messages = false;
+  subscription->is_cft_supported = false;
   return subscription;
 fail:
   if (topic_str) {
@@ -394,7 +395,7 @@ rmw_subscription_get_actual_qos(
 
 rmw_ret_t
 rmw_subscription_set_cft_expression_parameters(
-  const rmw_subscription_t * subscription,
+  rmw_subscription_t * subscription,
   const char * filter_expression,
   const rcutils_string_array_t * expression_parameters)
 {

diff --git a/rmw_connext_dynamic_cpp/src/functions.cpp b/rmw_connext_dynamic_cpp/src/functions.cpp
@@ -1182,6 +1182,7 @@ rmw_create_subscription(
   node_info->subscriber_listener->trigger_graph_guard_condition();
 
   subscription->can_loan_messages = false;
+  subscription->is_cft_supported = false;
   return subscription;
 fail:
   // Something has gone wrong, unroll what has been done.