Skip to content

Commit b82137c

Browse files
committed
No need to default-construct logger in the declaration
1 parent 13a1043 commit b82137c

File tree

1 file changed

+4
-7
lines changed

1 file changed

+4
-7
lines changed

generate_parameter_library_py/generate_parameter_library_py/jinja_templates/cpp/parameter_library_header

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,9 @@ struct StackParams {
105105
}
106106

107107
ParamListener(const std::shared_ptr<rclcpp::node_interfaces::NodeParametersInterface>& parameters_interface,
108-
rclcpp::Logger logger, std::string const& prefix = "") {
109-
logger_ = std::move(logger);
110-
prefix_ = prefix;
108+
rclcpp::Logger logger, std::string const& prefix = "")
109+
: logger_{std::move(logger)},
110+
prefix_{prefix} {
111111
if (!prefix_.empty() && prefix_.back() != '.') {
112112
prefix_ += ".";
113113
}
@@ -220,10 +220,7 @@ struct StackParams {
220220
std::shared_ptr<rclcpp::node_interfaces::OnSetParametersCallbackHandle> handle_;
221221
std::shared_ptr<rclcpp::node_interfaces::NodeParametersInterface> parameters_interface_;
222222

223-
// rclcpp::Logger cannot be default-constructed
224-
// so we must provide a initialization here even though
225-
// every one of our constructors initializes logger_
226-
rclcpp::Logger logger_ = rclcpp::get_logger("{{namespace}}");
223+
rclcpp::Logger logger_;
227224
std::mutex mutable mutex_;
228225
};
229226

0 commit comments

Comments
 (0)