Skip to content

Use TRACETOOLS_ prefix for tracepoint-related macros#1058

Merged
mjcarroll merged 1 commit intoros2:rollingfrom
christophebedard:christophebedard/add-namespace-prefix-tracetools-macros
Jun 8, 2023
Merged

Use TRACETOOLS_ prefix for tracepoint-related macros#1058
mjcarroll merged 1 commit intoros2:rollingfrom
christophebedard:christophebedard/add-namespace-prefix-tracetools-macros

Conversation

@christophebedard
Copy link
Copy Markdown
Member

@christophebedard christophebedard commented Apr 9, 2023

Part of ros2/ros2_tracing#18

Requires ros2/ros2_tracing#56

Signed-off-by: Christophe Bedard bedard.christophe@gmail.com

@christophebedard
Copy link
Copy Markdown
Member Author

Holding off on this until after Iron branches off from Rolling (April 24th).

@christophebedard christophebedard marked this pull request as draft April 11, 2023 17:28
@christophebedard christophebedard marked this pull request as ready for review May 3, 2023 20:11
@christophebedard christophebedard force-pushed the christophebedard/add-namespace-prefix-tracetools-macros branch from 268236e to 2747c64 Compare May 3, 2023 20:11
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.com>
@christophebedard christophebedard force-pushed the christophebedard/add-namespace-prefix-tracetools-macros branch from 2747c64 to 88a23ff Compare June 2, 2023 17:18
@christophebedard
Copy link
Copy Markdown
Member Author

This is ready for a review now that the Iron dust has settled a bit.

@christophebedard
Copy link
Copy Markdown
Member Author

@mjcarroll would you be able to review/approve this one?

@mjcarroll mjcarroll merged commit c43c7ab into ros2:rolling Jun 8, 2023
@christophebedard christophebedard deleted the christophebedard/add-namespace-prefix-tracetools-macros branch June 8, 2023 17:39
danthony06 pushed a commit to danthony06/rcl that referenced this pull request Jun 14, 2023
Signed-off-by: Christophe Bedard <bedard.christophe@gmail.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.

2 participants