-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Fix PCL_DEPRECATED
usage in doxygen for a proper Deprecation List in documentation
#3905
Conversation
registration/include/pcl/registration/correspondence_rejection_poly.h
Outdated
Show resolved
Hide resolved
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.
This is 3 PRs:
- Change for the title (sed, doxygen, etc.)
- Using the macro instead of warning/doxygen
- Changes in
pcl_visualizer.cpp
The commits' names aren't sufficient to address the related-but-different scopes here? I'll split this up, sure. I've had this for 3 iterations now, though (link):
Why doesn't it like the macro? I'll think about it in the morning. |
I stashed the changes in visualizer as they'll most likely come around again while clang-doxy-ing [visualization] |
PCL_DEPRECATED
usage in doxygen for a proper Deprecation List in documentation
Close #3825
from #3825 (comment)
I also added a sed invocation to deal with the whitespace, thanks Sergey.