[Bug gas/21934] gas build fails when using gcc 7

[bneumeier at gmail dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=21934

Brett Neumeier <bneumeier at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #10331|0                           |1
        is obsolete|                            |

--- Comment #12 from Brett Neumeier <bneumeier at gmail dot com> ---
Created attachment 10346
  --> https://sourceware.org/bugzilla/attachment.cgi?id=10346&action=edit
improved fix suggested by Martin Sebor

I filed a GCC bug (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81840) and
mentioned that here, but currently the GCC bugzilla denies that bug exists.
Weird.

Anyway, in a comment on that bug, Martin Sebor suggested adding a "hh" prefix
to the snprintf conversion specifier, to explicitly inform GCC that the integer
conversion corresponds to a char argument. That does indeed avoid the
false-positive warning message, and seems compliant with the defined behavior
of printf, so maybe it makes sense to apply that change here in binutils?

-- 
You are receiving this mail because:
You are on the CC list for the bug.