From: Felipe Balbi > Sent: 10 December 2015 15:14 > "Geyslan G. Bem" <geyslan@xxxxxxxxx> 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@xxxxxxxxx> > > --- > > I guess you didn't get previous comment in time; let's split this per > driver so different maintainers can pick their parts. I also suspect that gcc will optimise out the redundant checks as well. David -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html