On Fri, Jun 22, 2012 at 1:22 PM, Dmitri Gribenko <[email protected]> wrote: > On Fri, Jun 22, 2012 at 12:52 PM, Matt Beaumont-Gay > <[email protected]> wrote: >> On Fri, Jun 22, 2012 at 12:45 PM, Dmitri Gribenko <[email protected]> >> wrote: >>> On Fri, Jun 22, 2012 at 11:47 AM, Matt Beaumont-Gay >>> <[email protected]> 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 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
