[clang-tools-extra] [clang-tidy] use upper case letters for bool conversion suffix (PR #102831)

Carlos Galvez via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 16 13:25:11 PDT 2024


carlosgalvezp wrote:

> An inconsistency is that readability-uppercase-literal-suffix only handles l and u by default whereas the check here also handles f

Then that check should be updated to handle the F suffix as well. 

It doesn't make sense to have different style for different suffixes: either all lower case or all upper case.

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


More information about the cfe-commits mailing list