Add more bugprone-* clang-tidy checks, Part B #6380

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.

Sign up for GitHub

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

Jump to bottom

Open

gnawme wants to merge 1 commit into PointCloudLibrary:master from gnawme:norm.evangelista/add-further-bugprone-checks-part-b

+570 −637

Contributor

gnawme commented Dec 23, 2025

Migrate #6368, Part B including apps changes to manage the size of the PR.

Add 4 additional bugprone clang-tidy checks and applied fixes

Summary

These checks improve code safety and correctness by:

Preventing subtle bugs from dangling references
Ensuring consistent namespace declarations
Fixing incorrect container erase patterns
Eliminating side effects in assertions


          Add more bugprone-I clang-tidy checks, Part B

30086ce

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet