Refactor package using C++17 API #2596
Unanswered
RyuYamamoto
asked this question in
Ideas
Replies: 1 comment 5 replies
-
@RyuYamamoto Yes, it's super nice!!! |
Beta Was this translation helpful? Give feedback.
5 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi,
Now C++17 is available, so I'm thinking that I should try to replace code that could be replaced with latest function.
For example, I think the part that loads multiple PCDs with pointcloud_map_loader could be replaced with std::filesystem available from C++17.
https://github.com/autowarefoundation/autoware.universe/blob/main/map/map_loader/src/pointcloud_map_loader/pointcloud_map_loader_node.cpp#L72-L89
I haven't looked at the other parts that could be replaced yet, but is it no problem if I send PullRequest if I found something that could be fixed?
(If you know other parts in package of autoware.universe that could replaced , I would appreciate it if you could tell me.)
Beta Was this translation helpful? Give feedback.
All reactions