[Top][All Lists]

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

Re: [Qemu-devel] [PATCH v2 1/3] bitops: fix types

From: Blue Swirl
Subject: Re: [Qemu-devel] [PATCH v2 1/3] bitops: fix types
Date: Thu, 12 Jul 2012 20:21:28 +0000

On Wed, Jul 11, 2012 at 12:49 PM, Kevin Wolf <address@hidden> wrote:
> Am 08.07.2012 21:22, schrieb address@hidden:
>> From: Blue Swirl <address@hidden>
>> Use 'unsigned int' for bit numbers instead of 'unsigned long' or
>> 'int'. Adjust asserts.
>> Signed-off-by: Blue Swirl <address@hidden>
> I haven't followed the original discussion and therefore don't know what
> the controversy is about (nor do I feel like reading it up), but if
> there is no consensus, I would expect even more than already for normal
> patches that the commit message doesn't only state the obvious change,
> but also the reasons for the change.
> This message isn't much different from the famous "i++; /* increase i by
> one */" code comment.

The message could be improved by vast amounts, but in my view it is
sufficient for such a simple change.

> Kevin

reply via email to

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