hi,
On Fri, Sep 5, 2008 at 11:31 PM, Stanislav Malyshev <[EMAIL PROTECTED]> wrote:
> Hi!
>
> Attached is the patch that does the following for php.ini parser:
> 1. \{LETTER} is literal inside "" now, whatever the letter is
> 2. \" is " inside "", with exception of:
> 3. foo="bar\"{NEWLINE} is parsed as bar\ - i.e. \ is allowed before closing
> " but only if the line ends there (otherwise it's impossible to know what
> the intent was).
>
> That should make Windows paths work just fine (unless you have some really
> weird ones that I didn't think about) with double quotes.
That's exactly why I am reluctant to change what Jani did. It is
easier to deal with this minor breakage in 5.3.0 and then have a
consistent and clean configuration system. Continuing to add some
exceptions while hoping that "no weird things" are used is a bad idea
as well. There is a kind of breakage, but it is obvious and will be
directly seen as either the include_path will be incomplete (app
fails), an extension will be not be loaded, etc.
Jani even suggested to add _fatal_ error. The lack of error messages
was one of the main arguments against this feature.
--
Pierre
http://blog.thepimp.net | http://www.libgd.org
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php
