Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:17714
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Date: Tue, 9 Aug 2005 20:55:42 +0200
Reply-To: Marcus Boerger <helly@php.net>
Message-ID: <937135535.20050809205542@marcus-boerger.de>
To: Andi Gutmans <andi@zend.com>, Michael Wallner <mike@php.net>
Cc: internals@lists.php.net
In-Reply-To: <77.16.04646.4DF98F24@pb1.pair.com>
References: <34.85.04646.D7C98F24@pb1.pair.com>
	<77.16.04646.4DF98F24@pb1.pair.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] Re: RFC: internal class' static properties
From: helly@php.net (Marcus Boerger)

Hello Michael,

Tuesday, August 9, 2005, 2:19:25 PM, you wrote:

> I wrote:

>> Therefore I'd suggest changing that so that they'll be
>> initialized and destructed for each request, which means
>> that there will need to be some form of hooks, which

> Let me change that part... :)

> There's no need for any "hooks" as there's already
> RINIT() and RSHUTDOWN(), so I guess all I'm proposing
> is not handling static properties of internal classes
> persitently.

class consts should be easy and we two times had a patch for that.
And i definitively ask to support that since otherwise we flood our
namespace, see pdo, spl and the amount of defines there. With a peep
to pecl i see tons of more consts (especially in http).

regarding static props the solution is to define another hash table
in the class_entry that holdes the initial values for the static
variables. During minit these would be copied to the used hash table,
done. But even though the solution is pretty easy i do not know if we
really want to support that. We have default properties and all that
stuff already. Can't we simply check if default values for static
properties wouldn't do the job and if there is something missing in that?

Andi?

Best regards,
 Marcus