zinovy.nis added a comment.

> I think, it's 13, if you choose to remove stars, and 17 otherwise. The 
> difference is excessive spaces vs. required ones. Implementing proper logic 
> may be involved, but we can simplify it to something like "count all 
> non-space characters and a single space between words, but don't count spaces 
> around punctuation":

Isn't it a business of clang-format to determine the number of spaces between 
lexemes? IMO the solutuion you've provided for `GetTypeNameLength` is enough. 
Considering punctuation here is overkill :-)


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D45927



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to