Re: [Qemu-devel] [PATCH 2/8] block: bdrv_img_create(): add param_list argument

[Luiz Capitulino <lcapitulino@redhat.com>]

On Thu, 18 Oct 2012 13:57:45 +0200
Kevin Wolf <kwolf@redhat.com> wrote:

> Am 17.10.2012 21:35, schrieb Luiz Capitulino:
> > If set returns a copy of the parameter list used by the block driver
> > to create the new image.
> > 
> > Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
> > ---
> >  block.c    | 7 ++++++-
> >  block.h    | 3 ++-
> >  blockdev.c | 2 +-
> >  qemu-img.c | 2 +-
> >  4 files changed, 10 insertions(+), 4 deletions(-)
> > 
> > diff --git a/block.c b/block.c
> > index e95f613..254a5c2 100644
> > --- a/block.c
> > +++ b/block.c
> > @@ -4294,7 +4294,8 @@ bdrv_acct_done(BlockDriverState *bs, BlockAcctCookie *cookie)
> >  
> >  int bdrv_img_create(const char *filename, const char *fmt,
> >                      const char *base_filename, const char *base_fmt,
> > -                    char *options, uint64_t img_size, int flags)
> > +                    char *options, uint64_t img_size, int flags,
> > +                    QEMUOptionParameter **param_list)
> >  {
> >      QEMUOptionParameter *param = NULL, *create_options = NULL;
> >      QEMUOptionParameter *backing_fmt, *backing_file, *size;
> > @@ -4430,6 +4431,10 @@ int bdrv_img_create(const char *filename, const char *fmt,
> >      }
> >  
> >  out:
> > +    if (param_list && ret == 0) {
> > +        *param_list = append_option_parameters(NULL, param);
> > +    }
> 
> If you put this above the out: label, the ret == 0 check wouldn't be
> necessary.

I'll maintain it it there but will drop the ret check, so that we're
able to return the options on error too and thus keep the "Formatting"
message the way it's today.