<div class="gmail_quote">On Mon, May 28, 2012 at 3:29 AM, Yang Chen <span dir="ltr"><<a href="mailto:chenyang@cs.utah.edu" target="_blank">chenyang@cs.utah.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
Seems RecursiveASTVisitor has a bug which could cause duplicated visits to default arguments for a class template. For example, given the code below:<br>
<br>
class X;<br>
template<class T = X> class Y;<br>
template<class T> class Y {};<br>
<br>
The default argument 'X' will be visited twice. The attached patch fixed the problem. I also attached a small testcase. Thanks.</blockquote><div><br></div><div>Looks good from my side; perhaps Richard or Daniel can come up with more test cases?</div>
<div><br></div><div>Cheers,</div><div>/Manuel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
- Yang<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br>