omg-dds / dds-rtps

Validation of interoperability of products compliant with OMG DDS-RTPS standard.
https://www.omg.org/spec/DDSI-RTPS/
Other
11 stars 17 forks source link

Update the Content-Filtered Topic creation logic #15

Closed mitza-oci closed 4 months ago

mitza-oci commented 4 months ago

Before this change, the conditional code for EPROSIMA correctly used a distinct name for the CFT. This change expands that fix to apply to all implementations.

Also fixed a signed/unsigned warning -- I'm assuming sequence length() is unsigned on all implementations since it's true for C++ vector and for IDL-to-C++ mappings. [Edit: this turned out not to be true]