[clang] [clang-tools-extra] [clang-tidy][dataflow] Add `bugprone-null-check-after-dereference` check (PR #84166)

via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 19 02:34:21 PDT 2024


martinboehme wrote:

Some general comments:

*  Is this ready for the next round of review? I see you've pushed a change, but you remark in other comments that there are still changes left to do. Can you let us know when this is ready for review again?

*  When updating the PR, can you [use fixups instead of force-pushing](https://llvm.org/docs/GitHub.html#updating-pull-requests)? This makes it easier to see what you've changed.

*  This PR has a _lot_ of reviewers on it. I'd suggest keeping the ones who have made comments so far and removing the others. (They can always re-add themselves if they want to, but a PR shouldn't generally need this many reviewers.)

https://github.com/llvm/llvm-project/pull/84166


More information about the cfe-commits mailing list