public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "lopezibanez at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/26167] -Wconversion fails to detect signedness conversion from int to unsigned int in fuction call
Date: Mon, 18 Sep 2006 12:45:00 -0000	[thread overview]
Message-ID: <20060918124539.11320.qmail@sourceware.org> (raw)
In-Reply-To: <bug-26167-12152@http.gcc.gnu.org/bugzilla/>



------- Comment #13 from lopezibanez at gmail dot com  2006-09-18 12:45 -------
(In reply to comment #11)
> yes please. Actually I created my own patch for bringing the C++ frontend on
> ear with the C frontend, but I didn't submit it because it produced bazillions
> of (legal) warnings in the code I usually compile, too many to be useful. 
> 

I would like to take a look to your patch if that is possible.

A few things worth commenting on the wcoercion patch:

1) One of the development goals is to separate the original behaviour of
Wconversion in the C front end from the "other behaviour" of Wconversion
(slightly present on the C front end and pervasive in the C++ front end).
Therefore, Wconversion has been replaced by Wtraditional-conversion and
Wcoercion (the interesting one). The names are tentative, pending discussion by
GCC developers, and likely will change in the future. If you find all this
confusing, please read http://gcc.gnu.org/wiki/Wcoercion#Background. If you
work in the KDE project, it would be interesting (for me) to know your opinion
on how to handle the transition.

2) Wconversion option is not completely deleted by my patch, it still exists
for C++/Fortran/ObjC++ when the behaviour has not been modified and it didn't
conflict with Wtraditional-conversion or Wcoercion. Properly speaking, these
Wconversion warnings should be moved to Wcoercion. However, there is little
point on doing this until there is a definitive decision on the names of the
new options. Thus, to obtain the full set of coercion warnings in C++ you may
need to specify "-Wconversion -Wcoercion" (In short, Wconversion are the ones
that were already there undocumented, Wcoercion are new or modified).

3) There are a few issuess pending the review of GCC developers. For example,
warning messages could be more specific/informative.

4) I don't think we should pollute this bug report further. Thus, feel free to
write me directly. Much better would be to discuss things in public in the gcc
mail list: we may even get the attention of some GCC developer. 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=26167


  parent reply	other threads:[~2006-09-18 12:45 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-08  1:10 [Bug c++/26167] New: " kristian dot hermansen at gmail dot com
2006-02-08 11:36 ` [Bug c++/26167] " rguenth at gcc dot gnu dot org
2006-02-08 14:46 ` mueller at gcc dot gnu dot org
2006-02-08 15:32 ` gdr at integrable-solutions dot net
2006-02-26  6:40 ` kristian dot hermansen at gmail dot com
2006-08-21  5:41 ` pinskia at gcc dot gnu dot org
2006-09-16  1:08 ` lopezibanez at gmail dot com
2006-09-17 18:37 ` kristian dot hermansen at gmail dot com
2006-09-17 18:39 ` kristian dot hermansen at gmail dot com
2006-09-17 20:31 ` tbm at cyrius dot com
2006-09-18  8:16 ` lopezibanez at gmail dot com
2006-09-18 12:11 ` mueller at gcc dot gnu dot org
2006-09-18 12:22 ` lopezibanez at gmail dot com
2006-09-18 12:45 ` lopezibanez at gmail dot com [this message]
2006-10-16 15:37 ` mueller at gcc dot gnu dot org
2006-11-24  1:51 ` manu at gcc dot gnu dot org
2006-11-24  1:56 ` manu at gcc dot gnu dot org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20060918124539.11320.qmail@sourceware.org \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).