<div dir="ltr">On Thu, Jun 20, 2013 at 4:00 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Looks good.<br>
<div class="im"><br>
<br>
================<br>
Comment at: include/clang/ASTMatchers/ASTMatchers.h:2298<br>
@@ +2297,3 @@<br>
+  TOOLING_COMPILE_ASSERT(<br>
+    (llvm::is_base_of<Stmt, NodeType>::value) ||<br>
+    (llvm::is_base_of<Decl, NodeType>::value) ||<br>
----------------<br>
</div><div class="im">Manuel Klimek wrote:<br>
> Alexander Kornienko wrote:<br>
> > Is wrapping this in parens a way to shut up some diagnostic?<br>
> Yes. This is a macro, which otherwise takes the "," as the start of a new parameter.<br>
</div>Then I'd leave parentheses only around the whole logical expression and added a very short comment, so that nobody spends time trying to remove them.<br></blockquote><div><br></div><div style>Used only one set of parens, but I don't think commenting on how C++ works is a good idea (otherwise, where do we stop).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1009" target="_blank">http://llvm-reviews.chandlerc.com/D1009</a><br>
<br>
BRANCH<br>
  equalsboundnode<br>
<br>
ARCANIST PROJECT<br>
  clang<br>
</blockquote></div><br></div></div>