New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add how to fix the most vexing parse problem #541
Conversation
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
Can I suggest uniform initialization instead? It's more straightforward, and we required C++14 (for Foxy) or C++17 (for Galactic and Rolling). I'd still keep the comment, though updated to explain why uniform initialization must be used here. |
@clalancette sounds good to me, thanks for checking! |
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One more small item to fix in the comments, then I think this is good to go.
That said, I'm curious why rclcpp::QoS qos(rclcpp::KeepAll());
fails to compile. Naively, I expect it would. Should we file a bug in rclcpp
to follow-up and fix that?
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fine by me with green CI.
all green, going in. |
Ah, i do not have access on this repo 😢 @clalancette could you do me a favor? thanks 😄 |
address #540
Signed-off-by: Tomoya Fujita Tomoya.Fujita@sony.com