[PATCH] D71174: [clang-tidy] new check: bugprone-signed-char-misuse

[Tamás Zolnai via Phabricator via cfe-commits]

ztamas added a comment.

In D71174#1818883 <https://reviews.llvm.org/D71174#1818883>, @sylvestre.ledru wrote:

> In D71174#1774249 <https://reviews.llvm.org/D71174#1774249>, @ztamas wrote:
>
> > I run the new check on LibreOffice codebase with the option CharTypdefsToIgnore = "sal_Int8".
> >  The check produced 32 findings.
>
>
> Interesting. It found 31 issues on the Firefox code base!
>  I can share the list if you are interested!


Yes, share it, please! It would be very useful to see how the check works on another project.

In D71174#1818883 <https://reviews.llvm.org/D71174#1818883>, @sylvestre.ledru wrote:

> In D71174#1774249 <https://reviews.llvm.org/D71174#1774249>, @ztamas wrote:
>
> > I run the new check on LibreOffice codebase with the option CharTypdefsToIgnore = "sal_Int8".
> >  The check produced 32 findings.
>
>
> Interesting. It found 31 issues on the Firefox code base!
>  I can share the list if you are interested!





Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71174