From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:38356)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <armbru@redhat.com>) id 1avhWy-0001BK-Vc
	for qemu-devel@nongnu.org; Thu, 28 Apr 2016 04:42:41 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <armbru@redhat.com>) id 1avhWy-0005wB-3K
	for qemu-devel@nongnu.org; Thu, 28 Apr 2016 04:42:40 -0400
From: Markus Armbruster <armbru@redhat.com>
References: <1461750767-23273-1-git-send-email-kwolf@redhat.com>
	<1461750767-23273-7-git-send-email-kwolf@redhat.com>
	<5720E01A.1010909@redhat.com>
	<20160428082122.GA4474@noname.redhat.com>
Date: Thu, 28 Apr 2016 10:42:33 +0200
In-Reply-To: <20160428082122.GA4474@noname.redhat.com> (Kevin Wolf's message
	of "Thu, 28 Apr 2016 10:21:22 +0200")
Message-ID: <87mvoeruo6.fsf@dusky.pond.sub.org>
MIME-Version: 1.0
Content-Type: text/plain
Subject: Re: [Qemu-devel] [PATCH 06/17] bochs: Implement .bdrv_co_preadv()
 interface
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Kevin Wolf <kwolf@redhat.com>
Cc: Eric Blake <eblake@redhat.com>, famz@redhat.com, qemu-block@nongnu.org, sw@weilnetz.de, qemu-devel@nongnu.org, mreitz@redhat.com, stefanha@redhat.com

Kevin Wolf <kwolf@redhat.com> writes:

> 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 <kwolf@redhat.com>
>> > ---
>> >  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.

Four?  It comes out correctly with tab-width 8 for me.

You can have any tab stops you want as long as it's multiples of eight.

> 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.

We normally untabify the lines we touch anyway, and leave the rest
alone.