[PATCH] D25406: Fix doubled whitespace in modernize-use-auto fixit

[Aaron Ballman via cfe-commits]

aaron.ballman added inline comments.


================
Comment at: clang-tidy/modernize/UseAutoCheck.cpp:378
+      Lexer::getLocForEndOfToken(Range.getEnd(), 0, SM, Context->getLangOpts());
+  bool Whitespace = isWhitespace(*FullSourceLoc(Next, SM).getCharacterData());
+
----------------
Oye, this is deceptively expensive because you now have to go back to the actual source file for this information. That source file may live on a network share somewhere, for instance.

Can you use `Token::hasLeadingSpace()` instead?

Also, doesn't this still need to care about the `RemoveStars` option?


https://reviews.llvm.org/D25406