[ros-users] [Discourse.ros.org] [Next Generation ROS] Adding clang thread safety analysis for ROS2 core packages

[Emerson Knapp via Discourse.ros.org]

[quote="gbiggs, post:4, topic:7930"]

Yep, that makes sense. What Im wondering is does this have any impact on the analysis of threads that are annotated? If an annotated thread interacts with a non-annotated one, how trustworthy are the analysis results?

[/quote]



Good question - the way I understand it the annotations are more about data than the threads themselves - so if you annotate data as being guarded, the analysis will spit out warnings for other code/packages only if it is built with `-Wthread-safety`, and accesses that data unsafely (according to the the analysis rules). So we could keep "non-ported" code warning-free by not enabling that flag on it until it's been looked at specifically.











---

[Visit Topic](https://discourse.ros.org/t/adding-clang-thread-safety-analysis-for-ros2-core-packages/7930/6) or reply to this email to respond.