[Bug c/72858] incorrect fixit hints in -Wformat diagnostics

2016-10-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72858 --- Comment #5 from Martin Sebor --- Here's a slightly different but arguably even more compelling test case from x86_64-linux with ILP32 where wchar_t is a typedef for long for some strange reason yet wint_t is a typedef for int. Passing a wcha

[Bug c/72858] incorrect fixit hints in -Wformat diagnostics

2016-10-04 Thread msebor at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72858 --- Comment #4 from Martin Sebor --- I came across one subtle case that I think might be better handled differently than the others. The type of the expected argument to a %lc directive is wint_t, which is commonly int, but on some targets long

[Bug c/72858] incorrect fixit hints in -Wformat diagnostics

2016-09-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72858 --- Comment #3 from David Malcolm --- (In reply to David Malcolm from comment #2) > Author: dmalcolm > Date: Mon Sep 12 16:01:02 2016 > New Revision: 240095 The above commit fixes many of these issues; Martin identified some further improvements

[Bug c/72858] incorrect fixit hints in -Wformat diagnostics

2016-09-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72858 --- Comment #2 from David Malcolm --- Author: dmalcolm Date: Mon Sep 12 16:01:02 2016 New Revision: 240095 URL: https://gcc.gnu.org/viewcvs?rev=240095&root=gcc&view=rev Log: Fix suggestions for non-trivial Wformat type cases (PR c/72858) gcc/c-

[Bug c/72858] incorrect fixit hints in -Wformat diagnostics

2016-08-12 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72858 --- Comment #1 from David Malcolm --- Candidate patch posted here: https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01053.html

[Bug c/72858] incorrect fixit hints in -Wformat diagnostics

2016-08-11 Thread dmalcolm at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72858 David Malcolm changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|