[cfe-commits] r151934 - in /cfe/trunk: include/clang/Basic/DiagnosticGroups.td include/clang/Basic/DiagnosticSemaKinds.td test/Misc/serialized-diags.c test/Misc/warning-flags.c

[Matt Beaumont-Gay]

On Fri, Mar 2, 2012 at 12:23, Nico Weber <nicolasweber at gmx.de> wrote:
> Author: nico
> Date: Fri Mar  2 14:23:08 2012
> New Revision: 151934
>
> URL: http://llvm.org/viewvc/llvm-project?rev=151934&view=rev
> Log:
> Move int<->pointer conversion warnings behind -Wint-conversions.
>
> This is consistent with -Wbool-conversion. Let me know if you prefer
> a different flag name.
>
>
> Modified:
>    cfe/trunk/include/clang/Basic/DiagnosticGroups.td
>    cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
>    cfe/trunk/test/Misc/serialized-diags.c
>    cfe/trunk/test/Misc/warning-flags.c
>
> Modified: cfe/trunk/include/clang/Basic/DiagnosticGroups.td
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticGroups.td?rev=151934&r1=151933&r2=151934&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/Basic/DiagnosticGroups.td (original)
> +++ cfe/trunk/include/clang/Basic/DiagnosticGroups.td Fri Mar  2 14:23:08 2012
> @@ -27,6 +27,7 @@
>  def Availability : DiagGroup<"availability">;
>  def AutoImport : DiagGroup<"auto-import">;
>  def BoolConversions : DiagGroup<"bool-conversions">;
> +def IntConversions : DiagGroup<"int-conversions">;
>  def BuiltinRequiresHeader : DiagGroup<"builtin-requires-header">;
>  def CXXCompat: DiagGroup<"c++-compat">;
>  def CastAlign : DiagGroup<"cast-align">;
> @@ -278,7 +279,8 @@
>                             DiagGroup<"literal-conversion">,
>                             DiagGroup<"string-conversion">,
>                             DiagGroup<"sign-conversion">,
> -                            BoolConversions]>,
> +                            BoolConversions,
> +                            IntConversions]>,

This isn't your yak to shave, but I would really love it if we could
decide whether it should be "foo-conversion" or "foo-conversions" :)

-Matt