Make DomainParticipant::get_instance_handle return by value (#6149)

MiguelCompany · web-flow · commit d84c90ce1718 · 2026-01-15T12:03:30.000+01:00
Signed-off-by: Miguel Company &lt;miguelcompany@eprosima.com&gt;
diff --git a/include/fastdds/dds/domain/DomainParticipant.hpp b/include/fastdds/dds/domain/DomainParticipant.hpp
@@ -1159,7 +1159,7 @@ class DomainParticipant : public Entity
      *
      * @return InstanceHandle of this DomainParticipant.
      */
-    FASTDDS_EXPORTED_API const InstanceHandle_t& get_instance_handle() const;
+    FASTDDS_EXPORTED_API InstanceHandle_t get_instance_handle() const;
 
     // From here legacy RTPS methods.
 
diff --git a/src/cpp/fastdds/domain/DomainParticipant.cpp b/src/cpp/fastdds/domain/DomainParticipant.cpp
@@ -687,7 +687,7 @@ rpc::ServiceTypeSupport DomainParticipant::find_service_type(
     return impl_->find_service_type(service_type_name);
 }
 
-const InstanceHandle_t& DomainParticipant::get_instance_handle() const
+InstanceHandle_t DomainParticipant::get_instance_handle() const
 {
     return impl_->get_instance_handle();
 }
diff --git a/src/cpp/fastdds/domain/DomainParticipantImpl.cpp b/src/cpp/fastdds/domain/DomainParticipantImpl.cpp
@@ -775,7 +775,7 @@ IContentFilterFactory* DomainParticipantImpl::find_content_filter_factory(
     return &dds_sql_filter_factory_;
 }
 
-const InstanceHandle_t& DomainParticipantImpl::get_instance_handle() const
+InstanceHandle_t DomainParticipantImpl::get_instance_handle() const
 {
     return handle_;
 }
diff --git a/src/cpp/fastdds/domain/DomainParticipantImpl.hpp b/src/cpp/fastdds/domain/DomainParticipantImpl.hpp
@@ -653,7 +653,7 @@ class DomainParticipantImpl
     const rpc::ServiceTypeSupport find_service_type(
             const std::string& service_name) const;
 
-    const InstanceHandle_t& get_instance_handle() const;
+    InstanceHandle_t get_instance_handle() const;
 
     // From here legacy RTPS methods.
 

Original file line number	Diff line number	Diff line change
`@@ -1159,7 +1159,7 @@ class DomainParticipant : public Entity`
`1159`	`1159`	`*`
`1160`	`1160`	`* @return InstanceHandle of this DomainParticipant.`
`1161`	`1161`	`*/`
`1162`		`- FASTDDS_EXPORTED_API const InstanceHandle_t& get_instance_handle() const;`
	`1162`	`+ FASTDDS_EXPORTED_API InstanceHandle_t get_instance_handle() const;`
`1163`	`1163`
`1164`	`1164`	`// From here legacy RTPS methods.`
`1165`	`1165`
Original file line number	Diff line number	Diff line change
`@@ -687,7 +687,7 @@ rpc::ServiceTypeSupport DomainParticipant::find_service_type(`
`687`	`687`	`return impl_->find_service_type(service_type_name);`
`688`	`688`	`}`
`689`	`689`
`690`		`-const InstanceHandle_t& DomainParticipant::get_instance_handle() const`
	`690`	`+InstanceHandle_t DomainParticipant::get_instance_handle() const`
`691`	`691`	`{`
`692`	`692`	`return impl_->get_instance_handle();`
`693`	`693`	`}`
Original file line number	Diff line number	Diff line change
`@@ -775,7 +775,7 @@ IContentFilterFactory* DomainParticipantImpl::find_content_filter_factory(`
`775`	`775`	`return &dds_sql_filter_factory_;`
`776`	`776`	`}`
`777`	`777`
`778`		`-const InstanceHandle_t& DomainParticipantImpl::get_instance_handle() const`
	`778`	`+InstanceHandle_t DomainParticipantImpl::get_instance_handle() const`
`779`	`779`	`{`
`780`	`780`	`return handle_;`
`781`	`781`	`}`