<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Feb 21, 2011, at 5:43 PM, Douglas Gregor wrote:</div><blockquote type="cite"><div>On Feb 18, 2011, at 2:53 PM, John Wiegley wrote:<br><font class="Apple-style-span" color="#006312"><font class="Apple-style-span" color="#144FAE"><br></font></font><blockquote type="cite">--- a/lib/AST/ExprCXX.cpp<br></blockquote><blockquote type="cite">+++ b/lib/AST/ExprCXX.cpp<br></blockquote><blockquote type="cite">@@ -377,6 +377,14 @@ Stmt::child_iterator BinaryTypeTraitExpr::child_end() {<br></blockquote><blockquote type="cite"> return child_iterator();<br></blockquote><blockquote type="cite">}<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">+//ExpressionTraitExpr<br></blockquote><blockquote type="cite">+Stmt::child_iterator ExpressionTraitExpr::child_begin() {<br></blockquote><blockquote type="cite">+ return child_iterator();<br></blockquote><blockquote type="cite">+}<br></blockquote><blockquote type="cite">+Stmt::child_iterator ExpressionTraitExpr::child_end() {<br></blockquote><blockquote type="cite">+ return child_iterator();<br></blockquote><blockquote type="cite">+}<br></blockquote><br>These functions should return child iterators for &QueriedExpression and &QueriedExpression +1, respectively.<br></div></blockquote><div><br></div>Also, you'll need to implement children() instead of child_begin() / child_end(), and implementations this simple should really go in the header.</div><div><br></div><div>John.</div><br></body></html>