[cfe-commits] r163793 - in /cfe/trunk/include: clang-c/ clang/ARCMigrate/ clang/AST/ clang/CodeGen/ clang/Driver/ clang/Frontend/ clang/Lex/ clang/Rewrite/Core/ clang/Sema/ clang/Serialization/

[Douglas Gregor]

Sent from my iPhone

On Sep 13, 2012, at 12:07 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> On Thu, Sep 13, 2012 at 7:45 PM, Douglas Gregor <dgregor at apple.com> wrote:
>> 
>> On Sep 13, 2012, at 6:11 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>> 
>>> Author: gribozavr
>>> Date: Thu Sep 13 08:11:20 2012
>>> New Revision: 163793
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=163793&view=rev
>>> Log:
>>> Fix Doxygen misuse: refer to parameter names in paragraphs correctly (\arg is
>>> not what most people want -- it starts a new paragraph).
>> 
>> Is there a warning that goes along with this?
> 
> I was experimenting with warnings for unknown commands trying to find
> logic that would not be too noisy.  This was a one-off tool -- firing
> a warning for every \arg; it was very fruitful for LLVM codebase
> because \arg in paragraph is a misuse and \arg instead of \param is a
> style issue.
> 
> I could try to make a warning for \arg inside a paragraph.

>From the size of your patch, I suspect that would be a useful warning. 

> Dmitri
> 
> -- 
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/