[Top][All Lists]

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

Re: [Qemu-devel] [PATCH] Add support for r6040 NIC

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH] Add support for r6040 NIC
Date: Thu, 01 Sep 2011 09:39:25 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux)

Blue Swirl <address@hidden> writes:

> On Wed, Aug 31, 2011 at 4:06 PM, Anthony Liguori <address@hidden> wrote:
>> On 08/31/2011 09:35 AM, malc wrote:
>>> On Wed, 31 Aug 2011, Anthony Liguori wrote:
>>>> Upper case field names are not okay.  If you think coding style isn't
>>>> clear,
>>>> that's a bug in coding style.
>>> Sez hu? Coding style is garbage that should be thrown out of the window.
>>> As for looking, yeah, i'm looking at usb with it's lovely hungarian
>>> fields, should we stampede to "fix" it?
>>> If the one who's going to maintain the code is fine with whatever naming
>>> is used so be it.
>> No.  That's how we got into the coding style mess we're in in the first
>> place.
>> There's no benefit to going through and changing existing code but new code
>> needs to be consistent with the vast majority of code in the rest of the
>> tree.  It's about overall code base consistency and maintainability.
> I agree about importance of consistency, though I'd even go further
> and reformat globally. New code gets introduced based on copying old
> code so the pain goes on.

If we reformat globally (big if), then we better reformat to a
well-established style (such as linux kernel), not to this idiosyncratic
QEMU-only style.  Because the arguments about sharing and readability
apply across projects, too.

reply via email to

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