diff mbox

[v2,8/8] error: Add a 'error: ' prefix to error_report()

Message ID ba517a52c280fdd1768d2a16deeb5a5745dc99c3.1499453788.git.alistair.francis@xilinx.com
State New
Headers show

Commit Message

Alistair Francis July 7, 2017, 11:21 p.m. UTC 
As we don't regard error messages as a stable API the let's add a
'error: ' prefix to the original error_report() messages.

Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>
---

 util/qemu-error.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Philippe Mathieu-Daudé July 8, 2017, 4:36 a.m. UTC | #1 
On 07/07/2017 08:21 PM, Alistair Francis wrote:
> As we don't regard error messages as a stable API the let's add a
> 'error: ' prefix to the original error_report() messages.
> 
> Signed-off-by: Alistair Francis <alistair.francis@xilinx.com>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> ---
> 
>   util/qemu-error.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/util/qemu-error.c b/util/qemu-error.c
> index b555247e24..cf1eb8e8b9 100644
> --- a/util/qemu-error.c
> +++ b/util/qemu-error.c
> @@ -202,6 +202,7 @@ static void vreport(report_type type, const char *fmt, va_list ap)
>   
>       switch (type) {
>       case REPORT_TYPE_ERROR:
> +        error_printf("error: ");
>           break;
>       case REPORT_TYPE_WARNING:
>           error_printf("warning: ");
>
diff mbox

Patch

diff --git a/util/qemu-error.c b/util/qemu-error.c
index b555247e24..cf1eb8e8b9 100644
--- a/util/qemu-error.c
+++ b/util/qemu-error.c
@@ -202,6 +202,7 @@  static void vreport(report_type type, const char *fmt, va_list ap)
 
     switch (type) {
     case REPORT_TYPE_ERROR:
+        error_printf("error: ");
         break;
     case REPORT_TYPE_WARNING:
         error_printf("warning: ");