[cfe-commits] r133121 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td test/Sema/parentheses.c test/Sema/parentheses.cpp

Hans Wennborg hans at chromium.org
Thu Jun 16 07:27:00 PDT 2011


On Thu, Jun 16, 2011 at 2:05 AM, Chandler Carruth <chandlerc at gmail.com> wrote:
>  def warn_precedence_conditional : Warning<
> -  "?: has lower precedence than %0; %0 will be evaluated first">,
> +  "operator '?:' has lower precedence than '%0'; '%0' will be evaluated first">,
>   InGroup<Parentheses>;
>  def note_precedence_conditional_first : Note<
> -  "place parentheses around the ?: expression to evaluate it first">;
> +  "place parentheses around the '?:' expression to evaluate it first">;
>  def note_precedence_conditional_silence : Note<
> -  "place parentheses around the %0 expression to silence this warning">;
> +  "place parentheses around the '%0' expression to silence this warning">;

Looks good.

I basically copied that text from warn_precedence_bitwise_rel. Do you
think I should apply your change there as well for consistency?




More information about the cfe-commits mailing list