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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to