[clang-tools-extra] [clang-tidy] Rename 'cert-err52-cpp' to 'bugprone-avoid-setjmp-longjmp' (PR #159813)

via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 21 20:26:00 PDT 2025


zeyi2 wrote:

> After looking at the rule, I think we should place in "modernize" category: `modernize-avoud-setjmp-longjmp`. "bugprone" checks in general are expected to give only warnings for real problems: `setjmp` can be perfectly fine but it's not how exceptions should be modelled in C++, hence "modernize" category.

Thanks for reviewing, I've moved the check to `modernize-avoid-setjmp-longjmp` and kept the CERT link duplicated in both places, as requested.

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


More information about the cfe-commits mailing list