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

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


> 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
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180422/85f68471/attachment.html>


More information about the cfe-commits mailing list