ID:          32222
 Updated by:  [EMAIL PROTECTED]
 Reported By: philip at cornado dot com
 Status:      Bogus
 Bug Type:    Documentation problem
 PHP Version: Irrelevant
 New Comment:

Nice, it's been fixed in CVS :) How about NULL, should we change that
to ""?


Previous Comments:
------------------------------------------------------------------------

[2005-03-07 18:21:06] [EMAIL PROTECTED]

The script already searches for C macros. The on-line manual isn't
updated.

If you take a look at http://php.net/manual/en/ini.php you can see that
the default for include_path is ".;/path/to/php/pear".

If you find any strange value, just say ;)

------------------------------------------------------------------------

[2005-03-07 18:09:30] philip at cornado dot com

Description:
------------
For example:
http://php.net/manual/en/ini.core.php

include_path    PHP_INCLUDE_PATH        PHP_INI_ALL
doc_root        PHP_INCLUDE_PATH        PHP_INI_SYSTEM
user_dir        NULL    PHP_INI_SYSTEM
extension_dir   PHP_EXTENSION_DIR       PHP_INI_SYSTEM

Notice the constants being used as default values, this is not good. 

A couple years ago I posted the following proposal:
* http://marc.theaimsgroup.com/?l=phpdoc&m=105161194723229

In it contains various constants and their associated default values.
We need something similar to that for the generated ini table. Perhaps
check for default values if said value prefix is "PHP_" (except
safe_mode_allowed_env_vars) and have the generation script
(phpdoc/scripts/iniupdate/) find and insert the appropriate value OR
(but most likely not) simply hardcode them.





------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=32222&edit=1

Reply via email to