Re: [PATCH] mm/slab.c remove impossible <0 check - size_t is not signed - patch is against 2.6.1-rc1-mm2

From: Bill Davidsen
Date: Wed Jan 14 2004 - 20:18:40 EST


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@xxxxxxx>
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/