commit 6daf194dde15acda153b824299f36f12dfa527a6
author Markus Armbruster <armbru@redhat.com> Wed Jun 22 14:03:54 2011 +0200
committer Stefan Hajnoczi <stefanha@linux.vnet.ibm.com> Fri Jun 24 09:13:36 2011 +0100
tree e501eb043697f6ec77338ef8fe4cb5602177fdb0
parent db78ef5b0a93b16ad56b70a70e21b1c5e7d06ba8

Strip trailing '\n' from error_report()'s first argument

error_report() prepends location, and appends a newline.  The message
constructed from the arguments should not contain a newline.  Fix the
obvious offenders.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>

hw/milkymist-pfpu.c

diff --git a/hw/milkymist-pfpu.c b/hw/milkymist-pfpu.c
index 94e6315..306d1ce 100644
--- a/hw/milkymist-pfpu.c
+++ b/hw/milkymist-pfpu.c
@@ -301,7 +301,7 @@
     } break;
 
     default:
-        error_report("milkymist_pfpu: unknown opcode %d\n", op);
+        error_report("milkymist_pfpu: unknown opcode %d", op);
         break;
     }
 
@@ -358,7 +358,7 @@
                 /* decode at most MICROCODE_WORDS instructions */
                 if (i++ >= MICROCODE_WORDS) {
                     error_report("milkymist_pfpu: too many instructions "
-                            "executed in microcode. No VECTOUT?\n");
+                            "executed in microcode. No VECTOUT?");
                     break;
                 }
             }