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

Reply via email to