qemu-devel
[Top][All Lists]
Advanced

[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



reply via email to

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