<div class="gmail_quote">On Wed, Jan 4, 2012 at 5:05 PM, Matt Beaumont-Gay <span dir="ltr"><<a href="mailto:matthewbg@google.com">matthewbg@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Any objections to the new helper function in SourceManager? Also, any<br>
other interesting cases I should add tests for?<br></blockquote><div><br></div><div>This looks pretty good to me. If you're interested in other tests, statement expressions and comma operators maybe? It'd be good to test TEMP_FAILURE_RETRY, the case we care about, which is a statement expression IIRC.</div>
<div><br></div><div>Also, please test the source location before the Expr method, that method on Expr is very expensive.</div></div>