-
Notifications
You must be signed in to change notification settings - Fork 161
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
Runtime Interface Reflection: rcl #1025
Conversation
de39734
to
f12e5a7
Compare
0828ab6
to
e87aa66
Compare
bdfd5a5
to
51423ad
Compare
8d01f92
to
b71cb87
Compare
06745c8
to
318f951
Compare
aaf7e0d
to
dd7b5a1
Compare
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
9dde04c
to
0d6a665
Compare
Signed-off-by: methylDragon <methylDragon@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm aside from wanting to double check on some lingering TODO's. It's ok to have TODO's and merge this, I just want to check that they are still valid before merging.
Specifically there's some business related to type_description_interfaces__msg__TypeDescription
vs rosidl_runtime_c__type_description__TypeDescription
that I'm not sure about whether or not there is anything left to do or if the TODO's surrounding them can just be dropped.
Signed-off-by: methylDragon <methylDragon@gmail.com>
Signed-off-by: methylDragon <methylDragon@gmail.com>
* Implement runtime type typesupport struct Signed-off-by: methylDragon <methylDragon@gmail.com> * Implement first cut Signed-off-by: methylDragon <methylDragon@gmail.com> * Migrate to rosidl_dynamic_typesupport and update field IDs Signed-off-by: methylDragon <methylDragon@gmail.com> * Migrate to type description interfaces Signed-off-by: methylDragon <methylDragon@gmail.com> * Fix const Signed-off-by: methylDragon <methylDragon@gmail.com> * Refine signatures and use return types Signed-off-by: methylDragon <methylDragon@gmail.com> * Fix C linkage Signed-off-by: methylDragon <methylDragon@gmail.com> * Migrate methods to use return types Signed-off-by: methylDragon <methylDragon@gmail.com> * Lint Signed-off-by: methylDragon <methylDragon@gmail.com> * Support type hashes Signed-off-by: methylDragon <methylDragon@gmail.com> * Remove identifier alias Signed-off-by: methylDragon <methylDragon@gmail.com> * Change fini to destroy Signed-off-by: methylDragon <methylDragon@gmail.com> * Use create instead of init Signed-off-by: methylDragon <methylDragon@gmail.com> * Add rcutils conversion function and remove rmw interfaces Signed-off-by: methylDragon <methylDragon@gmail.com> * Clean up some comments Signed-off-by: methylDragon <methylDragon@gmail.com> --------- Signed-off-by: methylDragon <methylDragon@gmail.com>
* Implement runtime type typesupport struct Signed-off-by: methylDragon <methylDragon@gmail.com> * Implement first cut Signed-off-by: methylDragon <methylDragon@gmail.com> * Migrate to rosidl_dynamic_typesupport and update field IDs Signed-off-by: methylDragon <methylDragon@gmail.com> * Migrate to type description interfaces Signed-off-by: methylDragon <methylDragon@gmail.com> * Fix const Signed-off-by: methylDragon <methylDragon@gmail.com> * Refine signatures and use return types Signed-off-by: methylDragon <methylDragon@gmail.com> * Fix C linkage Signed-off-by: methylDragon <methylDragon@gmail.com> * Migrate methods to use return types Signed-off-by: methylDragon <methylDragon@gmail.com> * Lint Signed-off-by: methylDragon <methylDragon@gmail.com> * Support type hashes Signed-off-by: methylDragon <methylDragon@gmail.com> * Remove identifier alias Signed-off-by: methylDragon <methylDragon@gmail.com> * Change fini to destroy Signed-off-by: methylDragon <methylDragon@gmail.com> * Use create instead of init Signed-off-by: methylDragon <methylDragon@gmail.com> * Add rcutils conversion function and remove rmw interfaces Signed-off-by: methylDragon <methylDragon@gmail.com> * Clean up some comments Signed-off-by: methylDragon <methylDragon@gmail.com> --------- Signed-off-by: methylDragon <methylDragon@gmail.com>
Part of: ros2/ros2#1374
Implements new rosidl_message_type_support_t for runtime interface reflection