[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 1:22 PM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> 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.

Thanks!

-Matt