Re: [PHP4BETA] cvs: php4 /ext/standard/ basic_functions.c basic_functions.h php_noavail.c php_noavail.h

[email protected] (Hartmut Holzgraefe)
Newsgroups php.version4
Organization SIX Offene Systeme GmbH
Message-ID <[email protected]>
Zeev Suraski wrote:
> 
> zeev            Mon May 29 09:13:38 2000 EDT
> 
>   Removed files:
>     /php4/ext/standard  php_noavail.c php_noavail.h
> 
>   Modified files:
>     /php4/ext/standard  basic_functions.c basic_functions.h
>   Log:
>   There's really no need for an entire file for a one line function...
 
the reasons for putting in php_noavail.[ch] where

1) the mechanism behind warn_not_available() is not limit to
php4/ext/standard
   so i wasn't sure it should be in php4/ext/standard or in php4/

   puttig the implementation in a file of its own allows moving it
somewhere
   else without trouble

2) the prototype for warn_not_available() is needed by other modules
   (especially ext/imap) that i didn't check in yet
   that's what the .h file and it's inclusion into php.h was for
   
   removing this does still work in the repository as of now but breaks
   some other modules i didn't check in yet that make use of 
  
PHP_ALIAS(some_function_not_available_for_some_reason,warn_not_available,NULL)

   :(



> +       PHP_FE(warn_not_available,                      NULL)

warn_not_available is for internal use in alias entries for functions
not available and should not be visible to the user 
(as it does not provide any functionality suitable for userspace)


--
Hartmut Holzgraefe         [email protected]           http://www.six.de
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.