Hi Eric:
On Mon, Feb 09, 2009 at 09:27:03PM -0500, Eric Stewart wrote:
> http://wiki.php.net/rfc/newinis
Thanks for the work.
I noticed the actual directives in the files are the same. I guess you
just focused on the Quick Reference section and getting the comments set
up the way you want.
Line 488 in prod and 483 in dev need to have the link commented out.
error_reporting: should be E_ALL in production. Well written code
should not be generating notices, so notices indicate a problem that
needs fixing.
output_buffering: I agree with having it off in dev, since it encourages
cleaner programming practices. Relying on buffering can lead to getting
bitten when deploying to a server that has it off.
variables_order: They should be the same on dev and prod.
request_order: Seems like it should be the same.
track_errors: I like this on in all environments. Functions exist that
you need to get detailed error information but don't provide a way to get
at them in the code other than $php_errormsg. Your comment that it's
helpful for debugging dev servers undersells the this feature.
session.bug_compat_42 and session.bug_compat_warn: The grammar in your
comments needs cleaning up. Hmm, perhaps all of the comments do :), I
haven't looked. Anyway, the worst offense is the use of a double
negative: "It's not recommended that you do not use this feature on
production servers."
Thanks,
--Dan
--
T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y
data intensive web and database programming
http://www.AnalysisAndSolutions.com/
4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
