[PATCH] Fix bool expression special case.

Daniel Jasper djasper at google.com
Thu Mar 27 03:44:27 PDT 2014



================
Comment at: lib/Format/TokenAnnotator.cpp:48
@@ +47,3 @@
+    Contexts.back().InTemplateArgument =
+        Left->Previous != NULL && Left->Previous->Tok.isNot(tok::kw_template);
+
----------------
I think, this deserves a comment. After all, even in template<...> we are in a template argument, just not in one that concerns us. Or maybe, this is a matter of getting the names right. Is a template argument the same as a template parameter? Or is one used at the definition and one at the declaration?


http://llvm-reviews.chandlerc.com/D3187

BRANCH
  fix-template-bool

ARCANIST PROJECT
  clang



More information about the cfe-commits mailing list