search

ros2 / rosidl_dds

Apache License 2.0
6 stars 7 forks source link

support sequences of upper bounded strings #38

Closed dirk-thomas closed 7 years ago

dirk-thomas commented 7 years ago

This replaces #37 and fixes #36. The second commit only updates the code style a bit. The second commit fixes a problem with unbounded sequences since the .idl parsers seem to not support >>. Changing the generated code to > > works.

The numerous new test failures are unrelated to this patch (all for rmw_fastrtps) and like a result of a recent change in FastRTPS (https://github.com/eProsima/Fast-RTPS/commit/9d655e0b02affdc86218f8ccaabf206efc5f4c64).

dirk-thomas commented 7 years ago

Since it has been a while I ran another set of CI jobs:

Since they look good I will go ahead and merge this.