[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] Fast Virtual Disk (FVD) Proposal Part 1
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 1/5] Fast Virtual Disk (FVD) Proposal Part 1 |
Date: |
Sat, 22 Jan 2011 09:02:45 +0000 |
On 20 January 2011 17:08, Stefan Weil <address@hidden> wrote:
> Yes, that's a problem with some parts of the old code.
> For files which you want to modify, you could remove
> the spaces with your script before applying your other
> modifications and create a separate patch which only
> removes the superfluous spaces.
(This kind of came up in the other thread about fixing
non-C89 comments. I don't have any particular interest in
this area of the qemu source so this is a general remark.)
I definitely dislike patches which change whitespace or
indentation for an entire file, even if they are standalone
"only fixing whitespace" patches; they make it much harder
to deal with forks and branches of qemu. I would prefer
it if we only fix whitespace, indent and bracing for lines
we're touching anyway.
-- PMM
- [Qemu-devel] [PATCH 2/5] Fast Virtual Disk (FVD) Proposal Part 2, (continued)
- [Qemu-devel] [PATCH 2/5] Fast Virtual Disk (FVD) Proposal Part 2, Chunqiang Tang, 2011/01/19
- [Qemu-devel] [PATCH 4/5] Fast Virtual Disk (FVD) Proposal Part 4, Chunqiang Tang, 2011/01/19
- [Qemu-devel] [PATCH 3/5] Fast Virtual Disk (FVD) Proposal Part 3, Chunqiang Tang, 2011/01/19
- [Qemu-devel] [PATCH 5/5] Fast Virtual Disk (FVD) Proposal Part 5, Chunqiang Tang, 2011/01/19
- Re: [Qemu-devel] [PATCH 1/5] Fast Virtual Disk (FVD) Proposal Part 1, Christoph Hellwig, 2011/01/20
Re: [Qemu-devel] [PATCH 1/5] Fast Virtual Disk (FVD) Proposal Part 1, Anthony Liguori, 2011/01/21