Hi Alex,<div><br></div><div><div class="gmail_quote">On Thu, Apr 12, 2012 at 6:35 PM, Alexander Kornienko <span dir="ltr"><<a href="mailto:alexfh@google.com" target="_blank">alexfh@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Richard,<br><br>Thanks for the review. The next patch is attached. It addresses issues you pointed out, and doxygen comments style (thanks to Chandler).<div>

<br></div><div>I've also added a separate diagnostic message for wrong attributes used on statements, and a test for it. These require some work in Parser::ParseCXX0XAttributeSpecifier (which I didn't touch yet).</div>


<div><br></div><div>Please, review this patch.</div></blockquote><div><br></div><div>Thanks for all your effort! This patch looks like a great start. I've checked it in as r154723, with a couple of minor tweaks:</div>
<div> * I shortened the wording of the diagnostic a little.</div><div> * I removed the ProhibitAttributes when parsing declaration statements -- these are rejected in the appropriate circumstances in ParseDeclaration.</div>
<div><br></div><div>-- Richard</div></div></div>