[...]
> Suppose everything is working correctly and the pointer never is NULL.
> Then it really doesn't matter whether you check or not; the loss in code
> speed and size is completely negligible (except maybe deep in some inner
> loop). But there is a loss in code clarity; when I see a check like that
> it makes me think, "What's that doing there? Can that pointer ever be
> NULL, or is someone just being paranoid?" Distractions of that sort don't
> help when trying to read code.
My personal paranoia when reading code goes the other way: How can I be
sure it won´t ever be NULL? Maybe it can't be now (and to find that out an
hour grepping around goes by), but the very next patch introduces the
possibility. Better have the function do an extra check, or make sure
somehow the assumption won't _ever_ be violated. But that is a large (huge,
even) cost, so...
-- Dr. Horst H. von Brand User #22616 counter.li.org Departamento de Informatica Fono: +56 32 654431 Universidad Tecnica Federico Santa Maria +56 32 654239 Casilla 110-V, Valparaiso, Chile Fax: +56 32 797513 - 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/