ros2 / rmw_iceoryx

rmw implementation for iceoryx
Apache License 2.0
144 stars 27 forks source link

change serialization to use `uint8_t` instead of `char` #19

Open Karsten1987 opened 4 years ago

Karsten1987 commented 4 years ago

The serialization payload should really be serialized with uin8_t instead of char to guarantee a consistent size of 8bit on all platforms. See here: https://github.com/ros2/rmw_iceoryx/blob/master/rmw_iceoryx_cpp/include/rmw_iceoryx_cpp/iceoryx_serialize.hpp#L33