[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/11] qemu-img: introduce qemu_img_handle_error
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 01/11] qemu-img: introduce qemu_img_handle_error() |
Date: |
Thu, 18 Apr 2013 13:44:15 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 16.04.2013 um 18:05 hat Pavel Hrdina geschrieben:
> Later in the patch series we will use this function few times.
> This will avoid of duplicating the code.
>
> Signed-off-by: Pavel Hrdina <address@hidden>
> ---
> qemu-img.c | 17 +++++++++++------
> 1 file changed, 11 insertions(+), 6 deletions(-)
>
> diff --git a/qemu-img.c b/qemu-img.c
> index 31627b0..dbacdb7 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -322,6 +322,16 @@ static int add_old_style_options(const char *fmt,
> QEMUOptionParameter *list,
> return 0;
> }
>
> +static int qemu_img_handle_error(const char *msg, Error *err)
> +{
> + if (error_is_set(&err)) {
> + error_report("%s: %s", msg, error_get_pretty(err));
> + error_free(err);
> + return 1;
> + }
> + return 0;
> +}
> +
> static int img_create(int argc, char **argv)
> {
> int c;
> @@ -400,13 +410,8 @@ static int img_create(int argc, char **argv)
>
> bdrv_img_create(filename, fmt, base_filename, base_fmt,
> options, img_size, BDRV_O_FLAGS, &local_err, quiet);
> - if (error_is_set(&local_err)) {
> - error_report("%s", error_get_pretty(local_err));
> - error_free(local_err);
> - return 1;
> - }
>
> - return 0;
> + return qemu_img_handle_error("qemu-img create failed", local_err);
> }
This makes a change to the error message that isn't mentioned in the
commit message. It should definitely be mentioned there, but I'm not
even sure if it's a good change. Today you get something like:
$ qemu-img create -f foo test.img
qemu-img: Unknown file format 'foo'
With the patch applied it becomes:
$ qemu-img create -f foo test.img
qemu-img: qemu-img create failed: Unknown file format 'foo'
Does this add any useful information or does it just make the error
message longer? I feel it's the latter.
Kevin
[Qemu-devel] [PATCH 03/11] savevm: update bdrv_snapshot_find() to find snapshot by id or name, Pavel Hrdina, 2013/04/16
[Qemu-devel] [PATCH 04/11] qapi: Convert delvm, Pavel Hrdina, 2013/04/16