> You are right, in all cases we know of, the code does the correct checks :-).
> But I vaguely remember fixing bugs where the code did not properly check for SP_TYPE.
> It's just that there might be bad code somewhere, and then a NULL exception is that much easier to kill. The performance hit is not an issue I think, and is something that is easy to measure.

I’ll add that as you suggested, nevermind ;) .

Regards,

Markus