On Fri, 02 Sep 2011 12:42:26 +0100, Rob Richards <rricha...@cdatazone.org> wrote:

I finally got a chance to look at the code. Any reason you didn't just
manage everything in a per request state similar to the other global
function usage? You might even be able to get away with only
initializing it on the first call to setting the loader and you already
are performing actions in the RSHUTDOWN so no real additional overhead
taking that approach.

Because this setting is different from the others. It's a global setting, while the others are per-thread:



If I took the per-request approach, a request ending while other were active could deactivate the loader for those active. Additionally, setting this global option means threads with no PHP context will be affected too, so the preloader implementation has to determine whether it has a valid context.

Gustavo Lopes

PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to