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

Build time generation of fixed-size strings #758

Draft
wants to merge 13 commits into
base: rolling
Choose a base branch
from

Conversation

MiguelCompany
Copy link
Contributor

This is part of a PoC towards achieving the goals of ros2/rclcpp#2201

  • On rosidl_runtime_cpp, a new class has been added that behaves as a bounded string but can be considered POD by the underlying middleware.
  • On rosidl_generator_cpp, processing of a new cdr_plain annotation has been introduced, so string members with that annotation will be generated using the new class.
  • On rosidl_adapter, strings can be annotated with this new annotation depending on the contents of a YAML file specified in the environment variable ROS2_TYPE_FIXED_SIZE_FILE.

MiguelCompany and others added 13 commits June 13, 2023 16:42
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Miguel Company <MiguelCompany@eprosima.com>
Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com>
Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com>
Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com>
Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com>
Signed-off-by: Eduardo Ponz <eduardoponz@eprosima.com>
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

Successfully merging this pull request may close these issues.

None yet

2 participants