<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 11, 2013 at 10:06 AM, Warren Hunt <span dir="ltr"><<a href="mailto:whunt@google.com" target="_blank" class="cremed">whunt@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Yes, I should have noted there are two warnings that I haven't implemented.  It's much easier to drop them into the builder than it is to add them to Sema, although it makes more logical sense to put them in Sema (which would be a different patch).  I'm okay with either, are there any strong votes?</blockquote>
</div><br>I really feel like this warning is better done is Sema. Notably, the AST should actually preserve all of the information necessary to implement this warning, and not putting it in the (weird) place of *inside* the AST implementation is a great way of ensuring this remains true.</div>
</div>