Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

strange overwritten error with #69

Closed
wjwwood opened this issue Jun 1, 2016 · 3 comments
Closed

strange overwritten error with #69

wjwwood opened this issue Jun 1, 2016 · 3 comments

Comments

@wjwwood
Copy link
Member

wjwwood commented Jun 1, 2016

For example on my machine, while testing alpha 6:

% two_node_pipeline__rmw_fastrtps_cpp
[rmw|error_handling.c:93] error string being overwritten: not implemented, at /Users/william/ros2_ws/src/eProsima/ROS-RMW-Fast-RTPS-cpp/rmw_fastrtps_cpp/src/functions.cpp:1768
Published message with value: 0, and address: 0x7FFA898004B0
 Received message with value: 0, and address: 0x7FFA898004B0
Published message with value: 1, and address: 0x7FFA885AE6D0
 Received message with value: 1, and address: 0x7FFA885AE6D0
Published message with value: 2, and address: 0x7FFA885AE6D0
 Received message with value: 2, and address: 0x7FFA885AE6D0
Published message with value: 3, and address: 0x7FFA885AE6D0
 Received message with value: 3, and address: 0x7FFA885AE6D0
^Csignal_handler(2)

It is because of this nonsense that I had in the fastrtps function to get the graph guard condition:

https://github.com/eProsima/ROS-RMW-Fast-RTPS-cpp/blob/master/rmw_fastrtps_cpp/src/functions.cpp#L1768-L1769

I'm working on a fix.

@wjwwood
Copy link
Member Author

wjwwood commented Jun 1, 2016

Fix: ros2/rmw_fastrtps#32

@jacquelinekay
Copy link
Contributor

Your fix got merged, so we can close this, right?

@wjwwood wjwwood closed this as completed Jun 1, 2016
@wjwwood
Copy link
Member Author

wjwwood commented Jun 1, 2016

Yup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants