On Mon, Oct 1, 2012 at 9:46 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Sat, Sep 29, 2012 at 3:44 AM, Richard Trieu <<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>> wrote:<br>
> Do a little cleaning for the self reference visitor.<br>
> -Allow Sema to do more processing on the initial Expr before checking it.<br>
> -Remove the special conditions in HandleExpr()<br>
> -Move the code so that only one call site is needed.<br>
> -Removed the function from Sema and only call it locally.<br>
> -Warn on potentially evaluated reference variables, not just those cast to<br>
> r-values.<br>
> -Update tests.<br>
><br>
> This new version will catch the following cases:<br>
> int &a(a);<br>
> int &a{a};<br>
> int a(a);<br>
> int a{a};<br>
<br>
</div></div>Awesome, thanks for fixing!<br>
<span class="HOEnZb"><font color="#888888"><br>
 - Hans<br>
</font></span></blockquote></div><div><br></div>r164951