[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