================
Comment at: include/clang/ASTMatchers/Dynamic/VariantValue.h:56
@@ +55,3 @@
+  ///
+  /// It follows a similar interface than VariantMatcher itself.
+  class Payload {
----------------
Manuel Klimek wrote:
> isn't it similar ... as?
Done.

================
Comment at: include/clang/ASTMatchers/Dynamic/VariantValue.h:57
@@ +56,3 @@
+  /// It follows a similar interface than VariantMatcher itself.
+  class Payload {
+  public:
----------------
Manuel Klimek wrote:
> It looks like Payload basically carries around the type information. Thus, 
> I'd call it TypeInfo?
Well, it carries the type information, plus the actual value.


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

BRANCH
  variant_matcher

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

Reply via email to