httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dean Gaudet <dgau...@arctic.org>
Subject Re: cvs commit: apache-2.0/src/modules/mpm/winnt winnt.c
Date Tue, 12 Oct 1999 09:17:04 GMT


On Tue, 12 Oct 1999, Greg Stein wrote:

> A test for NULL seems a lot cheaper than a function call. If we have to
> call ap_null_cleanup, then yes... it is an extra test at cleanup
> registration time. I'd still be willing to pay it (it isn't much), but
> then again I won't be bent out of shape if I must use ap_null_cleanup.

you pay the test for NULL on every iteration, even though it's rarely ever
going to be true (the common cleanup case doesn't have nulls -- it's the
child cleanups which has nulls occasionally).  so the function call
doesn't happen.

Dean


Mime
View raw message