[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 9:02 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:
> Author: gribozavr
> Date: Fri Jun 22 11:02:55 2012
> New Revision: 159001
>
> URL: http://llvm.org/viewvc/llvm-project?rev=159001&view=rev
> Log:
> Add a warning about almost-Doxygen trailing comments: //< and /*< ... */
>
> Added:
>    cfe/trunk/test/Sema/doxygen-comments.c
> Modified:
>    cfe/trunk/include/clang/Basic/DiagnosticGroups.td
>    cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
>    cfe/trunk/lib/Sema/Sema.cpp
>    cfe/trunk/test/Analysis/retain-release.m
>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=159001&r1=159000&r2=159001&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original)
> +++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Fri Jun 22 11:02:55 2012
> @@ -57,6 +57,7 @@
>  def : DiagGroup<"disabled-optimization">;
>  def : DiagGroup<"discard-qual">;
>  def : DiagGroup<"div-by-zero">;
> +def Doxygen : DiagGroup<"doxygen">;
>  def EmptyBody : DiagGroup<"empty-body">;
>  def ExtraTokens : DiagGroup<"extra-tokens">;
>
>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td?rev=159001&r1=159000&r2=159001&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td (original)
> +++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Fri Jun 22 11:02:55 2012
> @@ -5665,5 +5665,10 @@
>   "definition of %0 must be imported before it is required">;
>  }
>
> +let CategoryName = "Documentation Issue" in {
> +def warn_not_a_doxygen_trailing_member_comment : Warning<
> +  "not a Doxygen trailing comment">, InGroup<Doxygen>;

Hi Dmitri,

Should this be DefaultIgnore? (I think yes.)

-Matt