This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: C PATCH to rectify warning for character types (PR c/23087)

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Joseph Myers <joseph at codesourcery dot com>, Marek Polacek <polacek at redhat dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 8 Jan 2016 18:54:32 +0100
Subject: Re: C PATCH to rectify warning for character types (PR c/23087)
Authentication-results: sourceware.org; auth=none
References: <20160107171127 dot GK31604 at redhat dot com> <alpine dot DEB dot 2 dot 10 dot 1601072117140 dot 31194 at digraph dot polyomino dot org dot uk>

On 01/07/2016 10:19 PM, Joseph Myers wrote:

I don't think it's desirable to raise the warning for this case under
different conditions from the warning for other signedness cases.  The
targets do differ in signedness - it's just that the difference is between
"plain" and "signed" or "plain" and "unsigned", not between signed and
unsigned.  Maybe the warning message should be more specific in this case,
but not a less-specific "incompatible" which is what this patch would
achieve.

I was going to voice the same opinion yesterday but forgot to hit Send.If you consider signedness of char a tri-state, then there's nothingwrong with the warning message.



Bernd

Follow-Ups:
- Re: C PATCH to rectify warning for character types (PR c/23087)
  - From: Marek Polacek

References:
- C PATCH to rectify warning for character types (PR c/23087)
  - From: Marek Polacek
- Re: C PATCH to rectify warning for character types (PR c/23087)
  - From: Joseph Myers

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]