Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

received WStrings messages from FastRTPS incorrect #364

Closed
dirk-thomas opened this issue Jul 31, 2019 · 2 comments
Closed

received WStrings messages from FastRTPS incorrect #364

dirk-thomas opened this issue Jul 31, 2019 · 2 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@dirk-thomas
Copy link
Member

Bug report

Required Info:

Steps to reproduce issue

Enable the test which publishes a WStrings message with non-ASCII characters using FastRTPS and subscribes using Connext by removing the following condition which skips the test atm: ros2/system_tests@49cdb2f#diff-02dc43db6f966560e7ee414c298df21eR193

Expected behavior

The test passes as it does with all other cross-vendor combinations.

Actual behavior

The test fails because the message returned by Connext doesn't match the expectation. The members which have default values with non-ASCII values are not terminated correctly. E.g. the value of the wstring_value_default2 member doesn't end after 12 characters but is actually 19 characters long including the value of the following member wstring_value_default3.

Additional information

The test passes in all other variations:

  • all platforms: FastRTPS -> FastRTPS, Connext -> Connext, Connext -> FastRTPS
  • FastRTPS -> Connext when the subscriber side is not running on macOS
@dirk-thomas dirk-thomas added bug Something isn't working help wanted Extra attention is needed labels Jul 31, 2019
@dirk-thomas
Copy link
Member Author

@neil-rti Maybe you can provide some feedback if this is a known issue.

@clalancette
Copy link
Contributor

Closing, since with Foxy now being End-of-Life, this repository is no longer used.

@clalancette clalancette closed this as not planned Won't fix, can't repro, duplicate, stale Jun 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants