<div dir="ltr"><br><div>It does look like the unevaluated context is the only thing missing at this point.  It seems reasonable to me that the unevaluated context might be useful to other attributes, outside of thread safety, so we could consider adding an extra bit to Attr in Attr.td.</div>
<div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 4, 2013 at 10:40 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Drat!  I knew it was too good to be true.  ;-)  Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
On Wed, Sep 4, 2013 at 1:28 PM, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:<br>
><br>
> On 4 Sep 2013 10:16, "Aaron Ballman" <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>><br>
>> With the refactorings that have been happening for attribute parsing,<br>
>> it seems this piece of custom parsing logic for thread safety analysis<br>
>> can go away.  The only real modification here is the updated test case<br>
>> which provides a definition for two functions; I wanted to make sure I<br>
>> was not subverting the intention of the test.<br>
><br>
> Sadly, you are. We need the unevaluated context around the expression<br>
> parsing here. But I suspect most of the duplication can still be factored<br>
> out.<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>DeLesley Hutchins | Software Engineer | <a href="mailto:delesley@google.com" target="_blank">delesley@google.com</a> | 505-206-0315<br>
</div>