Re: Style question: Should one check for NULL pointers?

Horst von Brand (vonbrand@inf.utfsm.cl)
Fri, 11 Jul 2003 15:32:20 -0400


Alan Stern <stern@rowland.harvard.edu> said:

[...]

> 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/