As a high-level comment, I was skimming this patch and saw a bit of inconsistency in commenting I would like to see cleaned up for any patches going in:<div><br></div><div>New doxygen comments should generally use the more modern style:</div>
<div><br></div><div>/// \brief One sentence summary.</div><div>///</div><div>/// A more detailed description here.</div><div>void FooBarBaz(...) { ... }</div><div><br></div><div>Rather than the more redundant pattern:</div>
<div><br></div><div>/// FooBarBaz - One sentence summary.</div><div>///</div><div>/// A more detailed description here.</div><div>void FooBarBaz(...) { ... }</div><div><br></div><div>I also saw some doxygen comments w/ just a preceding double slash rather than the necessary triple slash.<br>
</div><div><br></div><div>Finally, it's useful to check for proper prose in the comments which are more than a name or some such. They should start with capital letters, use punctuation, and form complete sentences, etc.</div>
<div><br></div><div>Sorry for the pedantic reminders of the dark corners of the coding standards, just didn't want them to go unnoticed... I'll get out of your guys' hair now...</div><br><br><div class="gmail_quote">

On Fri Apr 06 00:26:29 GMT+200 2012, Richard Smith <<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>> wrote:
<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Apr 5, 2012 at 3:23 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote 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>