Need reviewers for ROSIDL related change

General
1

Hello, dear ROS developer community. I’m working on adding enums to ROSIDL (issue). We (at Apex.AI) already have this feature implemented and used for few months and would like to upstream it. This is my first contribution to core ROS packages, so I’m looking for some guidance. This is the main PR that has links to other PRs.

Questions:

  1. Most of the PRs depend on this small PR that just adds .idl file for testing. Could someone please review it?
  2. Would be great if someone can answer these questions that I asked in the issue and we continue there.

Thanks, Roman

cc @MichaelOrlov @lyle

4 Likes
2

Looks like when the C++ generator is run, you don’t get enum classes. Is there a reason that can’t be done? One of the key advantages to that would be compile time safety of not mixing enums.

3

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.