Messages in this thread | | | Date | Thu, 10 Dec 2015 12:16:12 -0300 | Subject | Re: [PATCH] usb: remove redundant conditions | From | "Geyslan G. Bem" <> |
| |
2015-12-10 12:13 GMT-03:00 Felipe Balbi <balbi@ti.com>: > "Geyslan G. Bem" <geyslan@gmail.com> 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@gmail.com> >> --- > > I guess you didn't get previous comment in time; let's split this per > driver so different maintainers can pick their parts.
Okeydokey. :) > > -- > balbi
-- Regards,
Geyslan G. Bem hackingbits.com
| |