================
Comment at: lib/AST/ASTDiagnostic.cpp:72
@@ +71,3 @@
+        = dyn_cast<TemplateSpecializationType>(Ty)) {
+      for (auto Arg : *TST) {
+          if (Arg.getKind() == TemplateArgument::Type)
----------------
Richard Smith wrote:
> Please don't use a `TemplateSpecializationType` as a container like this; 
> it's not notionally a container of template arguments. Instead, add an 
> `arguments` member function to `TemplateSpecializationType` to return the 
> arguments list.
The class already has `getArgs`, `getNumArgs` and `getArg` methods. But why 
does it have `begin` and `end` if not to be used as a container?

http://reviews.llvm.org/D3588



_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to