pcl
pcl copied to clipboard
PointCloudLibrary
Make const usage better in function signatures
Context
clang-tidy has been used successfully before to make wide changes. We can use it to improve the API in the whole repo. We need discussion on the potential fix and just applying the other fixes after setting up clang-tidy.
Describe the solution you'd like
Apply the following fixes:
Another potential fix:
@Prayag-raj-sharma That's great! For your information: we now have a clang-tidy GitHub Action (.github/workflows/clang-tidy.yml), and the file .clang-tidy could also be relevant. You could start with const-return-type, then continue with the other checks. Please make sure your pull requests are not too large, otherwise please split them.
where to make changes can you elaborate, i/ have read all about clang-tidy just need to know where to make changes.
@Prayag-raj-sharma You would add the check to .clang-tidy (twice, once for Checks and once for WarningsAsErrors). Then you create a pull request and fix all places where the GitHub Action complains.
You can work on it
On Tue, 21 Nov, 2023, 2:41 pm sibi-venti, @.***> wrote:
is it done? or can i work on it?
— Reply to this email directly, view it on GitHub https://github.com/PointCloudLibrary/pcl/issues/3968#issuecomment-1820512782, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUXBJ22AUDCICM5R3GFFSD3YFRV3VAVCNFSM4MO4JGI2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBSGA2TCMRXHAZA . You are receiving this because you were mentioned.Message ID: @.***>