[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] error: Avoid redudant error_propagate() usage
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH] error: Avoid redudant error_propagate() usage |
Date: |
Thu, 9 Jun 2016 18:59:19 -0300 |
User-agent: |
Mutt/1.6.1 (2016-04-27) |
On Thu, Jun 09, 2016 at 02:57:08PM -0600, Eric Blake wrote:
> On 06/09/2016 02:21 PM, Eduardo Habkost wrote:
> > This patch simplifies code that uses a local_err variable just to
> > immediately
> > use it for an error_propagate() call.
> >
> > Done using the following Coccinelle patch:
> >
>
> > +++ b/block.c
> > @@ -353,7 +353,6 @@ out:
> > int bdrv_create_file(const char *filename, QemuOpts *opts, Error **errp)
> > {
> > BlockDriver *drv;
> > - Error *local_err = NULL;
> > int ret;
> >
> > drv = bdrv_find_protocol(filename, true, errp);
> > @@ -361,8 +360,7 @@ int bdrv_create_file(const char *filename, QemuOpts
> > *opts, Error **errp)
> > return -ENOENT;
> > }
> >
> > - ret = bdrv_create(drv, filename, opts, &local_err);
> > - error_propagate(errp, local_err);
> > + ret = bdrv_create(drv, filename, opts, errp);
> > return ret;
>
> And I _know_ there's a Coccinelle recipe for further shortening this
> into 'return bdrv_create(...)' (since it was part of the tutorial class
> at last year's KVM Forum) - again, I don't know the actual syntax to use
> to get it, but it shouldn't be too hard to find in a web search. Fine
> as yet another followup patch.
This is an easy one. I will give it a try.
Probably v2 will become a 3-patch series.
--
Eduardo