Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:35398
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 64.233.184.232 as permitted sender)
DomainKey-Status: bad
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:references;
        b=dqYeECF2COW9qWgxML/Z1wSpKk42pMG9MRO5NMkp/efzuFFc9adqdCg3oopAZa8feCK3z+R0WvizBSizmdHfwtkT0A3FUjGFCz/YR8N3XOyq8hjJ523SBtpdElJtF8KgtU2mwRfEdolD1EGCHpEbdb5HqG9KvXqC4wA4SebybnE=
Message-ID: <fe05d1540802111331i5b831d60y7df604e17fd2244@mail.gmail.com>
Date: Mon, 11 Feb 2008 22:31:17 +0100
To: "Marcus Boerger" <helly@php.net>
Cc: "PHP Internals List" <internals@lists.php.net>
In-Reply-To: <971575499.20080209153315@marcus-boerger.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Content-Disposition: inline
References: <971575499.20080209153315@marcus-boerger.de>
Subject: Re: [PHP-DEV] [RFC] Conditional INI support
From: pierre.php@gmail.com ("Pierre Joye")

Hi Marcus,

Nice addition, it is really handy and it'll help to solve the php.iniS mess :)

On Feb 9, 2008 3:33 PM, Marcus Boerger <helly@php.net> wrote:
>   attached is a patch against 5.3 that brings three feature
> additions to INI parsing.
>
> 1) Ternary support for values
>
>   setting = ${value?1:2}

Sounds overkilled. Not like one will edit php.ini every day. I find
the classic if elseif endif clearer.

> If ${value} evaluates to true then setting becomes 1 otherwise 2.
> This cannot be nested and only works for values, not for setting names.
>
> 2) if-elif-else-endif support
>
> [IF ${value} == 1]
> setting = 1
> [ELIF ${value} == 2]
> setting = 2
> [ELSE]
> setting = 3
> [ENDIF]

Perfect if ELSEIF instead of ELIF (which I can't write correctly
anyway, ELSEIF comes automatically).

> This can be nested. Alternatively we could use apache style syntax that
> looks more like XML. The reason I used square brackets is that this is the
> smallest change to normal INI files.
>
> 3) Add more values to INI parsing, namely:
>
> ${php.version} = 50300
> ${php.debug} = 0
> ${php.zts} = 0
> ${php.sapi} = CLI
>
> Any comments?

I like it.

Thanks for your work!
-- 
Pierre
http://blog.thepimp.net | http://www.libgd.org