Well, by that logic you would be fine with:

#define TIME_CALL sapi_get_request_time(TSRMLS_C)

and in tsrm_virtual_cwd.c

#ifndef TIME_CALL
define TIME_CALL time(0)
#endif
...
t = CWDG(realpath_cache_ttl)?TIME_CALL:0;

Theoretically yes, that could work, but:


that makes it environment-dependent instead of explicitly-dependent, and
also way harder to debug.

Environmental side-effects like that are nasty.

Exactly. So I'd propose to think how to do it right - not excluding taking the whole cache & system-dependant filesystem things out of TSRM. They do not really related to the rest of TSRM and could be abstracted same way as open/output/error functions are abstracted.
--
Stanislav Malyshev, Zend Products Engineer
[EMAIL PROTECTED]  http://www.zend.com/

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to