[cfe-commits] r159001 - in /cfe/trunk: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/Sema.cpp test/Analysis/retain-release.m test/Sema/doxygen-comments.c

[Dmitri Gribenko]

On Fri, Jun 22, 2012 at 12:52 PM, Matt Beaumont-Gay
<matthewbg at google.com> wrote:
> On Fri, Jun 22, 2012 at 12:45 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
>> On Fri, Jun 22, 2012 at 11:47 AM, Matt Beaumont-Gay
>> <matthewbg at google.com> wrote:
>>> Hi Dmitri,
>>>
>>> Should this be DefaultIgnore? (I think yes.)
>>
>> Hi Matt,
>>
>> We'll be adding more warnings as we actually do the parsing.  So the
>> question is should all Doxygen warnings be DefaultIgnore.
>
> Probably also yes :)
>
> For context, this new warning turns up in ICU:
> icu/source/i18n/choicfmt.cpp:53:40: error: not a Doxygen trailing
> comment [-Werror,-Wdoxygen]
> #define LESS_THAN    ((UChar)0x003C)   /*<*/
>
> Doxygen warnings should be opt-in, just as use of Doxygen in a
> codebase at all is opt-in.

Done in r159026.

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>*/