Hi,
"Geyslan G. Bem" writes:
> 2015-12-10 12:13 GMT-03:00 Felipe Balbi :
>> "Geyslan G. Bem" writes:
>>
>>> This patch removes redundant conditions.
>>>
>>> - (!A || (A && B)) is the same as (!A || B).
>>> - (length && length > 5) can be reduced to a single evaluation.
>>>
>>> Caught by: cppc
2015-12-10 12:29 GMT-03:00 Geyslan G. Bem :
> 2015-12-10 12:17 GMT-03:00 David Laight :
>> From: Felipe Balbi
>>> Sent: 10 December 2015 15:14
>>> "Geyslan G. Bem" writes:
>>>
>>> > This patch removes redundant conditions.
>>> >
>>> > - (!A || (A && B)) is the same as (!A || B).
>>> > - (length
2015-12-10 12:17 GMT-03:00 David Laight :
> From: Felipe Balbi
>> Sent: 10 December 2015 15:14
>> "Geyslan G. Bem" writes:
>>
>> > This patch removes redundant conditions.
>> >
>> > - (!A || (A && B)) is the same as (!A || B).
>> > - (length && length > 5) can be reduced to a single evaluation.
From: Felipe Balbi
> Sent: 10 December 2015 15:14
> "Geyslan G. Bem" writes:
>
> > This patch removes redundant conditions.
> >
> > - (!A || (A && B)) is the same as (!A || B).
> > - (length && length > 5) can be reduced to a single evaluation.
> >
> > Caught by: cppcheck
> >
> > Signed-off-by:
2015-12-10 12:13 GMT-03:00 Felipe Balbi :
> "Geyslan G. Bem" writes:
>
>> This patch removes redundant conditions.
>>
>> - (!A || (A && B)) is the same as (!A || B).
>> - (length && length > 5) can be reduced to a single evaluation.
>>
>> Caught by: cppcheck
>>
>> Signed-off-by: Geyslan G. Bem
"Geyslan G. Bem" writes:
> This patch removes redundant conditions.
>
> - (!A || (A && B)) is the same as (!A || B).
> - (length && length > 5) can be reduced to a single evaluation.
>
> Caught by: cppcheck
>
> Signed-off-by: Geyslan G. Bem
> ---
I guess you didn't get previous comment in tim
2015-12-10 11:50 GMT-03:00 Felipe Balbi :
>
> Hi,
>
> "Geyslan G. Bem" writes:
>> This patch removes redundant conditions.
>>
>> - (!A || (A && B)) is the same as (!A || B).
>> - (length && length > 5) can be reduced to a single evaluation.
>>
>> Caught by: cppcheck
>>
>> Signed-off-by: Geyslan
Hi,
"Geyslan G. Bem" writes:
> This patch removes redundant conditions.
>
> - (!A || (A && B)) is the same as (!A || B).
> - (length && length > 5) can be reduced to a single evaluation.
>
> Caught by: cppcheck
>
> Signed-off-by: Geyslan G. Bem
Can you split this into one patch per driver, t
2015-12-10 7:44 GMT-03:00 Johan Hovold :
> On Wed, Dec 09, 2015 at 10:52:42PM -0300, Geyslan G. Bem wrote:
>> This patch removes redundant conditions.
>>
>> - (!A || (A && B)) is the same as (!A || B).
>> - (length && length > 5) can be reduced to a single evaluation.
>>
>> Caught by: cppcheck
>>
On Wed, Dec 09, 2015 at 10:52:42PM -0300, Geyslan G. Bem wrote:
> This patch removes redundant conditions.
>
> - (!A || (A && B)) is the same as (!A || B).
> - (length && length > 5) can be reduced to a single evaluation.
>
> Caught by: cppcheck
>
> Signed-off-by: Geyslan G. Bem
> ---
> driv
This patch removes redundant conditions.
- (!A || (A && B)) is the same as (!A || B).
- (length && length > 5) can be reduced to a single evaluation.
Caught by: cppcheck
Signed-off-by: Geyslan G. Bem
---
drivers/usb/gadget/udc/s3c-hsudc.c | 2 +-
drivers/usb/host/fhci-sched.c | 2 +-
11 matches
Mail list logo