[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-06-14 Thread rguenth at gcc dot gnu dot org
--- Comment #8 from rguenth at gcc dot gnu dot org 2009-06-14 14:42 --- In general patches need to be sent to gcc-patc...@gcc.gnu.org together with a ChangeLog entry following existing practice and a note how the patch was tested. Copyright assignment to the FSF is required for non-triv

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-06-14 Thread zorry at ume dot nu
--- Comment #7 from zorry at ume dot nu 2009-06-14 13:32 --- Gentoo's >=gcc-4.3.3 is build with -Wformat and -Wformat-security enable to. -- zorry at ume dot nu changed: What|Removed |Added -

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-03-24 Thread kees at outflux dot net
--- Comment #6 from kees at outflux dot net 2009-03-24 17:39 --- I'm trying to minimize the Ubuntu patch by getting changes accepted for the FSF GCC testsuite. I'm hoping to demonstrate that many of the changes are valid and represent stricter C coding, though none change the behavior o

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-03-24 Thread joseph at codesourcery dot com
--- Comment #5 from joseph at codesourcery dot com 2009-03-24 13:19 --- Subject: Re: New: overhaul printf formats and type casts in testsuite The testsuite in FSF GCC tests correct behavior of FSF GCC and code compiled with it. If correct behavior for Ubuntu GCC and code compiled

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-03-24 Thread pinskia at gcc dot gnu dot org
--- Comment #4 from pinskia at gcc dot gnu dot org 2009-03-24 07:28 --- (In reply to comment #3) > g++.old-deja/g++.pt/t39.C: It looks like ptr[0],[1],[2] are either int or > const > char (i.e. ptr is int* or const char*). %p doesn't make much sense in those > cases, so I opted for a ca

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-03-24 Thread kees at outflux dot net
--- Comment #3 from kees at outflux dot net 2009-03-24 07:10 --- g++.old-deja/g++.pt/t39.C: It looks like ptr[0],[1],[2] are either int or const char (i.e. ptr is int* or const char*). %p doesn't make much sense in those cases, so I opted for a cast. I'm not sure I followed your last se

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-03-23 Thread pinskia at gcc dot gnu dot org
--- Comment #2 from pinskia at gcc dot gnu dot org 2009-03-24 06:54 --- Let's look at the testcases: g++.dg/ext/align1.C: a run testcase, the printf can be removed in this case. g++.old-deja/g++.law/operators28.C: An assemble only testcase so the undefined behavior is not invoked at runt

[Bug testsuite/39537] overhaul printf formats and type casts in testsuite

2009-03-23 Thread kees at outflux dot net
--- Comment #1 from kees at outflux dot net 2009-03-24 06:34 --- Created an attachment (id=17530) --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=17530&action=view) testsuite updates for format strings and casts -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39537