From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O0LC6-00011v-FO for qemu-devel@nongnu.org; Fri, 09 Apr 2010 16:52:50 -0400 Received: from [140.186.70.92] (port=43296 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O0LA0-0000XS-76 for qemu-devel@nongnu.org; Fri, 09 Apr 2010 16:52:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O0L9M-0002uh-8f for qemu-devel@nongnu.org; Fri, 09 Apr 2010 16:50:06 -0400 Received: from moutng.kundenserver.de ([212.227.126.171]:53858) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O0L9L-0002uU-UD for qemu-devel@nongnu.org; Fri, 09 Apr 2010 16:50:00 -0400 From: Stefan Weil Date: Fri, 9 Apr 2010 22:49:51 +0200 Message-Id: <1270846193-5667-3-git-send-email-weil@mail.berlios.de> In-Reply-To: <1270846193-5667-1-git-send-email-weil@mail.berlios.de> References: <1270846193-5667-1-git-send-email-weil@mail.berlios.de> Subject: [Qemu-devel] [PATCH] m68k: Fix compiler warning (fprintf format string) List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: QEMU Developers When argument checking is enabled, gcc throws this error: error: format not a string literal and no format arguments The patch rewrites the statement to satisfy the compiler. It also removes a type cast which is not needed. Signed-off-by: Stefan Weil --- m68k-dis.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/m68k-dis.c b/m68k-dis.c index d38d5a2..7cd88f8 100644 --- a/m68k-dis.c +++ b/m68k-dis.c @@ -1104,7 +1104,7 @@ print_insn_arg (const char *d, { static const char *const cacheFieldName[] = { "nc", "dc", "ic", "bc" }; val = fetch_arg (buffer, place, 2, info); - (*info->fprintf_func) (info->stream, cacheFieldName[val]); + (*info->fprintf_func) (info->stream, "%s", cacheFieldName[val]); break; } @@ -1199,7 +1199,7 @@ print_insn_arg (const char *d, { static const char *const scalefactor_name[] = { "<<", ">>" }; val = fetch_arg (buffer, place, 1, info); - (*info->fprintf_func) (info->stream, scalefactor_name[val]); + (*info->fprintf_func) (info->stream, "%s", scalefactor_name[val]); } else { @@ -1804,7 +1804,7 @@ match_insn_m68k (bfd_vma memaddr, save_p = p; info->print_address_func = dummy_print_address; - info->fprintf_func = (fprintf_ftype) dummy_printer; + info->fprintf_func = dummy_printer; /* We scan the operands twice. The first time we don't print anything, but look for errors. */ -- 1.5.6.5