[Bug translation/90117] Replace %<%s%> with %qs

[msebor at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90117

Martin Sebor  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Martin Sebor  ---
Yes, it does, and most instances of this warning in GCC code should already be
cleaned up so I think we can resolve this bug as fixed.

$ cat pr90117.c && gcc -S -Wall -Wformat-diag pr90117.c
__attribute__ ((format (gcc_cdiag, 1, 2))) void 
cdiag (const char*, ...);

void f (void)
{
  cdiag ("%<%s%>", "");
}

pr90117.c: In function ‘f’:
pr90117.c:6:11: warning: quoted ‘%s’ directive in format; use ‘%qs’ instead
[-Wformat-diag]
6 |   cdiag ("%<%s%>", "");
  |   ^~

[Bug translation/90117] Replace %<%s%> with %qs

[egallager at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90117

Eric Gallager  changed:

   What|Removed |Added

 CC||msebor at gcc dot gnu.org

--- Comment #4 from Eric Gallager  ---
Martin, does -Wformat-diag catch this?

[Bug translation/90117] Replace %<%s%> with %qs

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90117

Jakub Jelinek  changed:

   What|Removed |Added

   Target Milestone|9.0 |9.2

--- Comment #3 from Jakub Jelinek  ---
GCC 9.1 has been released.

[Bug translation/90117] Replace %<%s%> with %qs

[roland.illig at gmx dot de]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90117

--- Comment #2 from Roland Illig  ---
(In reply to Martin Liška from comment #1)
> Makes sense, I'll integrate that to our linter.

I've already integrated that into the linter, see the latest attachment in bug
90176.

[Bug translation/90117] Replace %<%s%> with %qs

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90117

Martin Liška  changed:

   What|Removed |Added

   Target Milestone|--- |9.0

[Bug translation/90117] Replace %<%s%> with %qs

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90117

Martin Liška  changed:

   What|Removed |Added

 Status|UNCONFIRMED |ASSIGNED
   Last reconfirmed||2019-04-17
 CC||marxin at gcc dot gnu.org
   Assignee|unassigned at gcc dot gnu.org  |marxin at gcc dot 
gnu.org
 Ever confirmed|0   |1

--- Comment #1 from Martin Liška  ---
Makes sense, I'll integrate that to our linter.