Messages in this thread | | | From | Bill Davidsen <> | Subject | Re: [PATCH] mm/slab.c remove impossible <0 check - size_t is not signed - patch is against 2.6.1-rc1-mm2 | Date | Wed, 14 Jan 2004 20:13:16 -0500 |
| |
Joe Perches wrote: > On Thu, 2004-01-08 at 02:16, Paul Jackson wrote: > >>Jason asked: >> >>>Well, anything wrong in cleaning them [unsigned compare warnings] up? > > > In this case the warning is not unsigned compare but > "comparison of .* is always [true|false]". > > This sort of code generally makes me think someone did something wrong, > not just that the person added additional unnecessary checking.
Agreed, often muddy thinking.
-- bill davidsen <davidsen@tmr.com> CTO TMR Associates, Inc Doing interesting things with small computers since 1979 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |