Add TRACETOOLS_ prefix to tracepoint-related public macros

[christophebedard]

Closes #18

This adds a TRACETOOLS_ prefix to tracepoint-related public macros: TRACEPOINT(), TRACEPOINT_ENABLED(), and DO_TRACEPOINT(). Since this is part of the public API, I left the current macros there and marked them as deprecated by defining and calling an empty deprecated function. Downstream PRs:

• Use TRACETOOLS_ prefix for tracepoint-related macros rclcpp#2162
• Use TRACETOOLS_ prefix for tracepoint-related macros rcl#1058
• Use TRACETOOLS_ prefix for tracepoint-related macros rmw_fastrtps#686
• Use TRACETOOLS_ prefix for tracepoint-related macros rmw_cyclonedds#450

It also adds a leading underscore to internal macros, especially in public headers.

Signed-off-by: Christophe Bedard [email protected]

action-ros-ci-repos-override: https://gist.githubusercontent.com/christophebedard/0f809c29f26685eaee9365a2f5bfaa17/raw/43889baefe7f2aa9803a5204a9c23c8c88d9e899/ros2.repos

[christophebedard]

RMW is feature-frozen now, but if we merge this, then is it considered a bug fix to merge the corresponding rmw_* PRs? 🤔

[christophebedard]

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

[christophebedard]

This is ready for review.

[christophebedard]

• Linux
• Linux-aarch64
• Windows

[christophebedard]

@mjcarroll friendly ping for a review now that the Iron dust has settled a bit

[mjcarroll]

@christophebedard looks good, but we can probably use another round of CI as it has been over a month now.

[christophebedard]

New CI:

• Linux
• Linux-aarch64
• Windows

[christophebedard]

I see the same rclcpp.test_executors failure in a bunch of other tests jobs, so this looks good to me.

I can only merge this PR, not the other ones. @mjcarroll if I merge this one, can you merge the other ones at the same time?

[mjcarroll]

I see the same rclcpp.test_executors failure in a bunch of other tests, so this looks good to me.

Yep, I'm looking into that as well.

can you merge the other ones at the same time?

Yep, just shoot me a message when you are ready.