<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote">On Wed, Jun 13, 2012 at 7:58 PM, James Dennett <span dir="ltr"><<a href="mailto:jdennett@google.com" target="_blank">jdennett@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This makes SemaCodeComplete.cpp more Doxygen-friendly by changing the<br>
OBJC_AT_KEYWORD_NAME take a string literal argument where previously<br>
its second argument was an unquoted token; macro invocations such as<br>
OBJC_AT_KEYWORD_NAME(NeedAt,{) confuse Doxygen's parser.<br>
<br>
While I'm wary of changing code (rather than just comments) to work<br>
around Doxygen's limitations, in this case the change makes the code<br>
more readable for human beings as well, and the macro derived no<br>
benefit from using the preprocessor's stringification operator, as<br>
it never has need of the unquoted token.<br>
<br>
I've also included a couple of trivial drive-by fixes to doc comments.</blockquote><div><br></div><div>LGTM. There is no functionality changed, so feel free to submit and folks can chime in on the commit if there are any concerns. </div>
</div></font></div>