[PATCH] D45927: [clang-tidy] [modernize-use-auto] Correct way to calculate a type name length for multi-token types

Zinovy Nis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 22 00:32:48 PDT 2018


zinovy.nis added a subscriber: malcolm.parsons.
zinovy.nis added a comment.

> I think spaces that will be removed should be counted - long long is 9.

I thought about it, but what about "long       long              int

- *     *     *"? Is it still 9?

I think it's a business of clang-format to remove excessive spaces, not of
clang-tidy.

вс, 22 апр. 2018 г. в 10:11, Malcolm Parsons via Phabricator <
reviews at reviews.llvm.org>:

> malcolm.parsons added a comment.
> 
> I think spaces that will be removed should be counted - long long is 9.
> 
> OTOH, MinTypeNameLength isn't likely to be set high enough for that to
>  matter.
> 
> Repository:
> 
>   rCTE Clang Tools Extra
> 
> https://reviews.llvm.org/D45927


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D45927





More information about the cfe-commits mailing list