[PATCH] D157188: [clang-tidy] Add bugprone-new-bool-conversion

Chandler Carruth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Aug 5 16:58:05 PDT 2023


chandlerc added a comment.

In D157188#4563143 <https://reviews.llvm.org/D157188#4563143>, @Eugene.Zelenko wrote:

> In D157188#4563139 <https://reviews.llvm.org/D157188#4563139>, @PiotrZSL wrote:
>
>> In D157188#4563105 <https://reviews.llvm.org/D157188#4563105>, @Eugene.Zelenko wrote:
>>
>>> Shouldn't C-style allocation be checked too? Same for custom allocators via configuration option.
>>
>> In theory it could, but in such case name of check would need to change...
>
> Something like `bugprone-resource-bool-conversion` should be generic enough.

Maybe `bugprone-allocation-bool-conversion`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157188/new/

https://reviews.llvm.org/D157188



More information about the cfe-commits mailing list