omtcyfz added inline comments.

================
Comment at: clang-rename/USRFinder.cpp:80
@@ -79,1 +79,3 @@
         TypeBeginLoc, 0, Context.getSourceManager(), Context.getLangOpts());
+    if (const auto *TemplateTypeParm = 
dyn_cast<TemplateTypeParmType>(Loc.getType())) {
+      return setResult(TemplateTypeParm->getDecl(), TypeBeginLoc, TypeEndLoc);
----------------
alexfh wrote:
> It's not common to use braces around single-line `if` bodies in LLVM/Clang 
> code.
I do that literally everywhere :D LLVM Code Style isn't against it and I thinks 
it is good in terms of readability.


https://reviews.llvm.org/D22853



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

Reply via email to