Sample ID incorrectly resets when SampleInfo/WriteParams goes out of thread scope (#252)

* [#23141] First approach

Signed-off-by: danipiza <dpizarrogallego@gmail.com>

* [#23141] Updated SampleIdentity and WriteParams

Signed-off-by: danipiza <dpizarrogallego@gmail.com>

* [#23141] Review changes

Signed-off-by: danipiza <dpizarrogallego@gmail.com>

* [#23141] Removed unnecessary files

Signed-off-by: danipiza <dpizarrogallego@gmail.com>

* Refs #23141. Add more copyctr

Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>

---------

Signed-off-by: danipiza <dpizarrogallego@gmail.com>
Signed-off-by: Ricardo González Moreno <ricardo@richiware.dev>
Co-authored-by: Ricardo González Moreno <ricardo@richiware.dev>

Author: Danipiza

fastdds_python/src/swig/fastdds/rtps/common/BinaryProperty.i

@@ -17,7 +17,7 @@
 %}
 
 // Ignore overloaded constructor and methods that have no effect on target language
-%ignore eprosima::fastdds::rtps::BinaryProperty::BinaryProperty(BinaryProperty &&);
+%copyctor eprosima::fastdds::rtps::BinaryProperty;
 %ignore eprosima::fastdds::rtps::BinaryProperty::name(std::string &&);
 %ignore eprosima::fastdds::rtps::BinaryProperty::value(std::vector<uint8_t> &&);
 %ignore eprosima::fastdds::rtps::BinaryProperty::propagate() const;

fastdds_python/src/swig/fastdds/rtps/common/EntityId_t.i

@@ -28,6 +28,8 @@ long hash(const eprosima::fastdds::rtps::EntityId_t& id)
 }
 %}
 
+%copyctor eprosima::fastdds::rtps::EntityId_t;
+
 // Overloaded constructor ignored
 %ignore eprosima::fastdds::rtps::EntityId_t::EntityId_t(EntityId_t &&);
 %ignore eprosima::fastdds::rtps::operator <<(std::ostream&, const EntityId_t&);

fastdds_python/src/swig/fastdds/rtps/common/Guid.i

@@ -23,6 +23,8 @@ long hash(const eprosima::fastdds::rtps::GUID_t& guid)
 }
 %}
 
+%copyctor eprosima::fastdds::rtps::GUID_t;
+
 // SWIG does not support type conversion operators correctly unless converted to a normal method
 %rename(get_instance_handle) eprosima::fastdds::rtps::GUID_t::operator const InstanceHandle_t&;
 

fastdds_python/src/swig/fastdds/rtps/common/GuidPrefix_t.i

@@ -62,6 +62,8 @@ long hash(const eprosima::fastdds::rtps::GuidPrefix_t& prefix)
     $result = python_tuple;
 }
 
+%copyctor eprosima::fastdds::rtps::GuidPrefix_t;
+
 %ignore eprosima::fastdds::rtps::operator <<(std::ostream&, const GuidPrefix_t&);
 %ignore eprosima::fastdds::rtps::operator >>(std::istream&, GuidPrefix_t&);
 

fastdds_python/src/swig/fastdds/rtps/common/Locator.i

@@ -17,7 +17,7 @@
 %}
 
 // Ignore overloaded constructor and methods that have no effect on target language
-%ignore eprosima::fastdds::rtps::Locator_t::Locator_t(Locator_t&&);
+%copyctor eprosima::fastdds::rtps::Locator_t;
 %ignore eprosima::fastdds::rtps::operator <<(std::ostream&, const Locator_t&);
 %ignore eprosima::fastdds::rtps::operator >>(std::istream&, Locator_t&);
 %ignore eprosima::fastdds::rtps::operator ==(const Locator_t&, const Locator_t&);

fastdds_python/src/swig/fastdds/rtps/common/OriginalWriterInfo.i

@@ -16,8 +16,11 @@
 #include "fastdds/rtps/common/OriginalWriterInfo.hpp"
 %}
 
+%copyctor eprosima::fastdds::rtps::OriginalWriterInfo;
+
 // Ignore overloaded constructor and methods that have no effect on target language
 %ignore eprosima::fastdds::rtps::OriginalWriterInfo::OriginalWriterInfo(OriginalWriterInfo &&);
+%ignore eprosima::fastdds::rtps::OriginalWriterInfo::OriginalWriterInfo(GUID_t &&, SequenceNumber_t &&);
 %ignore eprosima::fastdds::rtps::OriginalWriterInfo::original_writer_guid(GUID_t &&);
 %ignore eprosima::fastdds::rtps::OriginalWriterInfo::original_writer_guid() const;
 %ignore eprosima::fastdds::rtps::OriginalWriterInfo::sequence_number(SequenceNumber_t &&);

fastdds_python/src/swig/fastdds/rtps/common/Property.i

@@ -17,7 +17,7 @@
 %}
 
 // Ignore overloaded constructor and methods that have no effect on target language
-%ignore eprosima::fastdds::rtps::Property::Property(Property &&);
+%copyctor eprosima::fastdds::rtps::Property;
 %ignore eprosima::fastdds::rtps::Property::name(std::string &&);
 %ignore eprosima::fastdds::rtps::Property::value(std::string &&);
 %ignore eprosima::fastdds::rtps::Property::propagate() const;

fastdds_python/src/swig/fastdds/rtps/common/SampleIdentity.i

@@ -17,7 +17,7 @@
 %}
 
 // Ignore overloaded constructor and methods that have no effect on target language
-%ignore eprosima::fastdds::rtps::SampleIdentity::SampleIdentity(SampleIdentity &&);
+%copyctor eprosima::fastdds::rtps::SampleIdentity;
 %ignore eprosima::fastdds::rtps::SampleIdentity::writer_guid(GUID_t &&);
 %ignore eprosima::fastdds::rtps::SampleIdentity::writer_guid() const;
 %ignore eprosima::fastdds::rtps::SampleIdentity::sequence_number(SequenceNumber_t &&);

fastdds_python/src/swig/fastdds/rtps/common/SequenceNumber.i

@@ -16,6 +16,7 @@
 #include "fastdds/rtps/common/SequenceNumber.hpp"
 %}
 
+%copyctor eprosima::fastdds::rtps::SequenceNumber_t;
 
 %ignore eprosima::fastdds::rtps::operator <<(std::ostream&, const SequenceNumber_t&);
 %ignore eprosima::fastdds::rtps::operator <<(std::ostream&, const std::vector<SequenceNumber_t>&);

fastdds_python/src/swig/fastdds/rtps/common/Time_t.i

@@ -21,6 +21,9 @@
 // We remove the one that is not used in the user API
 // We also remove all the related operators
 %rename(RTPSTime_t) eprosima::fastdds::rtps::Time_t;
+
+%copyctor eprosima::fastdds::rtps::Time_t;
+
 %ignore eprosima::fastdds::rtps::operator==(const Time_t&, const Time_t&);
 %ignore eprosima::fastdds::rtps::operator!=(const Time_t&, const Time_t&);
 %ignore eprosima::fastdds::rtps::operator<(const Time_t&, const Time_t&);
@@ -31,5 +34,7 @@
 %ignore eprosima::fastdds::rtps::operator-(const Time_t&, const Time_t&);
 %ignore eprosima::fastdds::rtps::operator<<(std::ostream&, const Time_t&);
 %ignore eprosima::fastdds::rtps::operator>>(std::istream&, Time_t&);
+%ignore eprosima::fastdds::rtps::Time_t::seconds() const;
+%ignore eprosima::fastdds::rtps::Time_t::fraction() const;
 
 %include <fastdds/rtps/common/Time_t.hpp>