[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

[Matt Beaumont-Gay]

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.

-Matt