[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] e1000: cleanup process_tx_desc
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] e1000: cleanup process_tx_desc |
Date: |
Tue, 4 Jun 2013 09:33:16 +0100 |
On 4 June 2013 08:34, Andrew Jones <address@hidden> wrote:
> I could send a v2 that fixes the 1 error and 2 warnings found in the context
> of this patch, but why? It's out of the scope of the patch (although I did
> use "cleanup" in the summary...), and it would hardly make a dent in this
> file's problems.
The idea is that we gradually bring the code closer into
line with QEMU's standards by (a) not allowing in new
code which doesn't follow the rules and (b) fixing old
code where it is in areas which a patch touches. This
gradually ratchets up the quality overall without being
huge "touch every line in a file" patches (which reduce
the functionality of git blame, among other things).
It really isn't a very onerous requirement in my opinion.
thanks
-- PMM