LGTM, subject to a couple of minor comments I've left on the codereview site.<br><br><div class="gmail_quote">On Tue, Apr 17, 2012 at 10:53 AM, Delesley Hutchins <span dir="ltr"><<a href="mailto:delesley@google.com">delesley@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thread safety analysis: split warnings into two groups: attribute warnings<br>
which are checked in the parser, and analysis warnings that require the<br>
full analysis.  This allows attribute syntax to be checked independently<br>
of the full thread safety analysis.  Also introduces a new warning for the<br>
case where a string is used as a lock expression; this allows the analysis<br>
to gracefully handle expressions that would otherwise cause a parse error.<br>
<br>
http://<a href="http://codereview.appspot.com/6051045/" target="_blank">http://codereview.appspot.com/6051045/</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
DeLesley Hutchins | Software Engineer | <a href="mailto:delesley@google.com">delesley@google.com</a> | <a href="tel:505-206-0315" value="+15052060315">505-206-0315</a><br>
</font></span></blockquote></div><br>