Skip to content

Conversation

@gnawme
Copy link
Contributor

@gnawme 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

  1. bugprone-assert-side-effect
  2. bugprone-dangling-handle
  3. bugprone-forward-declaration-namespace
  4. bugprone-inaccurate-erase

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

@gnawme gnawme force-pushed the norm.evangelista/add-further-bugprone-checks-part-b branch from 6e0167f to c26c4c6 Compare December 31, 2025 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant