qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 06/17] bochs: Implement .bdrv_co_preadv() interf


From: Kevin Wolf
Subject: Re: [Qemu-devel] [PATCH 06/17] bochs: Implement .bdrv_co_preadv() interface
Date: Thu, 28 Apr 2016 10:21:22 +0200
User-agent: Mutt/1.5.21 (2010-09-15)

Am 27.04.2016 um 17:51 hat Eric Blake geschrieben:
> On 04/27/2016 03:52 AM, Kevin Wolf wrote:
> > Signed-off-by: Kevin Wolf <address@hidden>
> > ---
> >  block/bochs.c | 46 +++++++++++++++++++++++++++++-----------------
> >  1 file changed, 29 insertions(+), 17 deletions(-)
> > 
> 
> >  static void bochs_close(BlockDriverState *bs)
> > @@ -267,7 +279,7 @@ static BlockDriver bdrv_bochs = {
> >      .instance_size = sizeof(BDRVBochsState),
> >      .bdrv_probe            = bochs_probe,
> >      .bdrv_open             = bochs_open,
> > -    .bdrv_read          = bochs_co_read,
> > +    .bdrv_co_preadv = bochs_co_preadv,
> >      .bdrv_close            = bochs_close,
> >  };
> 
> Alignment is funky here.  I'd rather just get rid of all the extra
> spaces, if that's easier than having half but not all of the = aligned.

Alignment is funky by definition when there are tabs involved and new
code follows the coding style and uses spaces instead. With a tab stop
of 4, this is aligned correctly.

The other option would be to just convert the whole struct to spaces,
but that's not really related to this patch and I think we're avoiding
pure style cleanup patches to keep things like 'git blame' useful.

Kevin

Attachment: pgpI31qQwzqKi.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]