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><div><br></div><div>Thanks!</div><div><br><div class="gmail_quote">On Fri, Apr 6, 2012 at 12:26 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div class="im">On Thu, Apr 5, 2012 at 3:23 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>Hi Alex,</div><div><br></div><div>I think the patch is generally looking pretty good. All of your TODOs seem like reasonable things to defer to later patches. I have some specific comments below (mostly formatting-related):</div>

</blockquote><div><br></div></div><div>Oh, and -- it would be great if you can come up with some way to test these changes. I appreciate that's going to be difficult since we don't accept any statement attributes yet, but you should at least be able to test that declaration attributes are rejected when applied to statements.</div>

</div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>
<div>Regards,</div><div>Alex</div></div>