Skip to content

Commit 73439fe

Browse files
authored
No need to default-construct logger in the declaration (#252)
1 parent 1ee5af1 commit 73439fe

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
}
@@ -233,10 +233,7 @@ struct StackParams {
233233
std::shared_ptr<rclcpp::node_interfaces::NodeParametersInterface> parameters_interface_;
234234
userParameterUpdateCB user_callback_;
235235

236-
// rclcpp::Logger cannot be default-constructed
237-
// so we must provide a initialization here even though
238-
// every one of our constructors initializes logger_
239-
rclcpp::Logger logger_ = rclcpp::get_logger("{{namespace}}");
236+
rclcpp::Logger logger_;
240237
std::mutex mutable mutex_;
241238
};
242239

0 commit comments

Comments
 (0)