Skip to content

Commit

Permalink
Modified structure member names to reflect the point made on the thre…
Browse files Browse the repository at this point in the history
…ad below.

ros-infrastructure/rep#385 (comment)

Signed-off-by: Shoji Morita <s-morita@esol.co.jp>
  • Loading branch information
smorita-esol committed Jan 26, 2024
1 parent 35af6fa commit 9c3af41
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion rclcpp/src/rclcpp/executors/multi_threaded_executor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,7 @@ default_thread_attr(rclcpp::ExecutorOptions const & options)
const rcutils_thread_attr_t * attr = std::find_if(
attrs_beg, attrs_end,
[&](const auto & attr) {
return attr.name == name;
return attr.tag == name;
});
if (attr != attrs_end) {
return rcpputils::ThreadAttribute(*attr);
Expand Down
2 changes: 1 addition & 1 deletion rclcpp/src/rclcpp/executors/single_threaded_executor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ default_thread_attr(rclcpp::ExecutorOptions const & options)
const rcutils_thread_attr_t * attr = std::find_if(
attrs_beg, attrs_end,
[&](const auto & attr) {
return attr.name == name;
return attr.tag == name;
});
if (attr != attrs_end) {
return rcpputils::ThreadAttribute(*attr);
Expand Down
4 changes: 2 additions & 2 deletions rclcpp/test/rclcpp/executors/test_multi_threaded_executor.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -160,11 +160,11 @@ constexpr char const * argv[] = {
"--ros-args",
"--thread-attrs-value",
R"(
- name: RCLCPP_EXECUTOR_MULTI_THREADED
- tag: RCLCPP_EXECUTOR_MULTI_THREADED
scheduling_policy: FIFO
priority: 10
core_affinity: []
- name: executor-1
- tag: executor-1
scheduling_policy: RR
priority: 20
core_affinity: [0]
Expand Down

0 comments on commit 9c3af41

Please sign in to comment.