Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:43077
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain oracle.com from 141.146.126.234 cause and error)
Message-ID: <499B079C.4010306@oracle.com>
Date: Tue, 17 Feb 2009 10:53:16 -0800
User-Agent: Thunderbird 2.0.0.19 (X11/20090105)
MIME-Version: 1.0
To: Eric Stewart <ericleestewart@gmail.com>
CC: internals@lists.php.net
References: <ce6b011c0902162302x477b5001i96ee667201cc103a@mail.gmail.com>
In-Reply-To: <ce6b011c0902162302x477b5001i96ee667201cc103a@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] New INIs, Round Two.
From: christopher.jones@oracle.com (Christopher Jones)


Eric,

Should uncommented parameters that seem to have the default value be
commented out?  For example "asp_tags" and "precision".  If the
parameters don't always have the same default value everywhere, should
they be documented in Quick Reference section as having a different
value to the default?

Eric Stewart wrote:
 > 4. I mistakenly had the development and production values of
 > allow_call_time_pass_reference reversed. This error has been
 > corrected.

I really think this should be Off in both cases to discourage use.
The doc http://www.php.net/ini.core says "This method is deprecated
and is likely to be unsupported in future versions of PHP/Zend."

 > 10. The production value of error_reporting has been changed to E_ALL |
 > ~E_DEPRECATED.

This should use '&', as Dave already pointed out on the list.

 > 12. The oci8.events and oci8.old_oci_close_semantics example values now use
 > the boolean constants.

Thanks.

 > 13. Many people have asked why the links to the online documentation for
 > each directive are specifically to the English version.

Regardless of the language issue, can the URLs consistently use "www"
instead of "us2"?  At the moment both occur.

Can the generic case in this come first:?

   ; 6. Windows directory (C:\windows or C:\winnt), or --with-config-file-path
   ; compile time option.

i.e change it to

   ; 6. The directory from the --with-config-file-path compile time
   ; option, or the Windows directory (C:\windows or C:\winnt)

The general documentation could mention the use of variables as seen
in ext/standard/tests/general_functions/parse_ini_basic.{phpt,data}:

   basicval = bar
   var1 = ${basicval}

The general documentation could mention that absolute paths to
extensions are (now) supported:

   extension=/path/to/extension.so

This should use "its" not "it's":

   ; PHP attempts to find and load this configuration from a number of locations.
   ; The following is a summary of it's search order:

The first "it's" below should be "its":

   ; php.ini-development is very similar to it's production variant, except it's
   ; much more verbose when it comes to errors.

This should be "its" in:

   ; php.ini-production contains settings which hold security, performance and
   ; best practices at it's core.

Ditto in:

   ; Turning on this setting and managing it's maximum buffer size can yield some

Ditto in:

   ;   Integer = Enables the buffer and sets it's maximum size in bytes.

Ditto in:

   ; this to 1 will cause PHP CGI to fix it's paths to conform to the spec.  A setting

There's an (existing) typo in this description, I guess "ignore
libjpeg warnings" was the intention:

   ; Tell the jpeg decode to libjpeg warnings and try to create
   ; a gd image.

Chris

-- 
Email: christopher.jones@oracle.com  Tel: +1 650 506 8630
Twitter:  http://twitter.com/ghrd    Free PHP Book: http://tinyurl.com/UGPOM