[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-11-02 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #19 from Jason Merrill jason at gcc dot gnu.org 2011-11-02 20:16:49 UTC --- Author: jason Date: Wed Nov 2 20:16:43 2011 New Revision: 180794 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=180794 Log: PR c++/50810

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-11-02 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Jason Merrill jason at gcc dot gnu.org changed: What|Removed |Added Status|REOPENED|RESOLVED

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-24 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #18 from Manuel López-Ibáñez manu at gcc dot gnu.org 2011-10-24 12:35:49 UTC --- Paolo, I am sorry to say it so bluntly but this solution is nonsense: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02160.html I know it is what Gabriel

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|ASSIGNED|RESOLVED

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread paolo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #11 from paolo at gcc dot gnu.org paolo at gcc dot gnu.org 2011-10-23 18:34:51 UTC --- Author: paolo Date: Sun Oct 23 18:34:45 2011 New Revision: 180343 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=180343 Log: /c-family

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread paolo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #13 from paolo at gcc dot gnu.org paolo at gcc dot gnu.org 2011-10-23 20:59:48 UTC --- Author: paolo Date: Sun Oct 23 20:59:43 2011 New Revision: 180348 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=180348 Log: /c-family

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|RESOLVED|REOPENED

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread paolo at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #15 from paolo at gcc dot gnu.org paolo at gcc dot gnu.org 2011-10-23 21:15:11 UTC --- Author: paolo Date: Sun Oct 23 21:15:08 2011 New Revision: 180349 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=180349 Log: /c-family

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #16 from Manuel López-Ibáñez manu at gcc dot gnu.org 2011-10-23 21:43:27 UTC --- (In reply to comment #14) Patch needs preliminary clean-ups in gcc/config, etc. It seems to me that enabling -Wc++0x-compat with -Wall is not a good

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-23 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #17 from Manuel López-Ibáñez manu at gcc dot gnu.org 2011-10-23 21:56:21 UTC --- Now that the patch needs to be recommitted, I would suggest another minor improvement. Please, remove the %qE so we do not print nonsense!

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-22 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added Status|NEW |ASSIGNED

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-22 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added CC|paolo.carlini at oracle dot |

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-22 Thread jason at redhat dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #9 from Jason Merrill jason at redhat dot com 2011-10-23 02:06:56 UTC --- I think that makes sense, yes. People can always specify -Wno-narrowing if they don't want it. paolo.carlini at oracle dot com gcc-bugzi...@gcc.gnu.org wrote:

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-22 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #10 from Paolo Carlini paolo.carlini at oracle dot com 2011-10-23 02:12:42 UTC --- Great. I'll post the patch tomorrow.

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-21 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Paolo Carlini paolo.carlini at oracle dot com changed: What|Removed |Added CC||jason at

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-21 Thread paolo.carlini at oracle dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #3 from Paolo Carlini paolo.carlini at oracle dot com 2011-10-21 22:57:31 UTC --- (detail: not sure about the 'if (!warn_narrowing) return;' at the beginning of check_narrowing: probably we don't want -Wno-narrowing to suppress the

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-21 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Manuel López-Ibáñez manu at gcc dot gnu.org changed: What|Removed |Added CC||manu at gcc

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-21 Thread manu at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #5 from Manuel López-Ibáñez manu at gcc dot gnu.org 2011-10-22 01:24:25 UTC --- (In reply to comment #4) (In reply to comment #3) (detail: not sure about the 'if (!warn_narrowing) return;' at the beginning of check_narrowing:

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-21 Thread jason at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 --- Comment #6 from Jason Merrill jason at gcc dot gnu.org 2011-10-22 04:38:15 UTC --- Makes sense to me.

[Bug c++/50810] c++0x-compat does not warn about narrowing conversions

2011-10-20 Thread redi at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50810 Jonathan Wakely redi at gcc dot gnu.org changed: What|Removed |Added Keywords||diagnostic