[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/8] usb: a trivial code change for more idio
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/8] usb: a trivial code change for more idiomatic writing style |
Date: |
Fri, 01 Aug 2014 05:42:41 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 |
Am 01.08.2014 05:32, schrieb Gonglei (Arei):
> Hi,
>
>> Subject: Re: [PATCH v2 2/8] usb: a trivial code change for more idiomatic
>> writing
>> style
>>
>> On 07/31/2014 08:32 PM, address@hidden wrote:
>>> From: Gonglei <address@hidden>
>>>
>>> Signed-off-by: Gonglei <address@hidden>
>>> ---
>>> hw/usb/dev-audio.c | 2 +-
>>> hw/usb/dev-mtp.c | 4 ++--
>>> hw/usb/hcd-ehci.c | 2 +-
>>> 3 files changed, 4 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/hw/usb/dev-audio.c b/hw/usb/dev-audio.c
>>> index bfebfe9..988f6cc 100644
>>> --- a/hw/usb/dev-audio.c
>>> +++ b/hw/usb/dev-audio.c
>>> @@ -371,7 +371,7 @@ static void output_callback(void *opaque, int avail)
>>> return;
>>> }
>>> data = streambuf_get(&s->out.buf);
>>> - if (NULL == data) {
>>> + if (data == NULL) {
>>
>> Wouldn't it be even more idiomatic as:
>>
>> if (!data) {
>>
>> Probably applies throughout your series.
>>
> OK, will do. Thanks!
Not so quick! You are free to use that in your patches, but please don't
change all code that way without the author's consent. Just like "equals
null" is a natural English way of reading, compared to "null equals
something", "not null" reads like a boolean expression to me, and even
worse while all valid C, "not strcmp" leads to mind-boggling inverted
logic...
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
[Qemu-devel] [PATCH v2 1/8] CODING_STYLE: Section about conditional statement, arei.gonglei, 2014/07/31
[Qemu-devel] [PATCH v2 6/8] spice: a trivial code change for more idiomatic writing style, arei.gonglei, 2014/07/31
[Qemu-devel] [PATCH v2 7/8] vl: a trivial code change for more idiomatic writing style, arei.gonglei, 2014/07/31
[Qemu-devel] [PATCH v2 5/8] a trivial code change for more idiomatic writing style, arei.gonglei, 2014/07/31
[Qemu-devel] [PATCH v2 4/8] isa-bus: a trivial code change for more idiomatic writing style, arei.gonglei, 2014/07/31
Re: [Qemu-devel] [PATCH v2 for-2.2 0/8] about Yoda conditions, Eric Blake, 2014/07/31