Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:42447
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain pooteeweet.org from 88.198.8.16 cause and error)
To: Marcus Boerger <helly@php.net>
In-Reply-To: <1715852784.20090102152003@marcus-boerger.de>
References: <01d401c8fe5e$6bc9d730$16b2a8c0@kartofel> <7EB12BC1-41F1-458C-8017-9AB4D910CAA5@pooteeweet.org> <FCBBD5E8B47B451A816FCB6B42C7477C@carla> <A5A404F4-2477-44C0-A1CC-EE911FB17BA4@pooteeweet.org> <48EFC1E8.5030702@zend.com> <5D4A905B-E361-409B-BD9E-E816482EDB8C@pooteeweet.org> <4D165E1668394935B126876B72D487CF@carla> <641f88250812311636l7fc8b29x34d4250763cdb903@mail.gmail.com> <49801FDC-2DFD-4A38-AC87-7AE9CBF893B9@pooteeweet.org> <1589625320.20090101145631@marcus-boerger.de> <fe05d1540901010855h159c3208t7e2e70b91cf71867@mail.gmail.com> <A8640977-6AEF-47AC-B029-052AE7638AD6@pooteeweet.org> <1715852784.20090102152003@marcus-boerger.de>
Message-ID: <C84CC2BC-E61A-48B2-976A-413A1BAA7D1C@pooteeweet.org>
Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes
Content-Transfer-Encoding: 7bit
Mime-Version: 1.0 (Apple Message framework v930.3)
Date: Fri, 2 Jan 2009 15:53:09 +0100
Cc: Pierre Joye <pierre.php@gmail.com>,
 "George Antoniadis" <george@noodles.gr>,
 "Timm Friebe" <thekid@thekid.de>,
 "Stanislav Malyshev" <stas@zend.com>,
  <internals@lists.php.net>,
 =?ISO-8859-1?Q?=22\Johannes_Schl=FCter=22?= <johannes@php.net>
Subject: Re: [PHP-DEV] __getStatic
From: mls@pooteeweet.org (Lukas Kahwe Smith)


On 02.01.2009, at 15:20, Marcus Boerger wrote:


> There are two things here. One is the __getStatic which I do not care
> for personally but which has the potential of requiring c level api
> changes. The other is 'static class' as in the patch provided by Lars.
> That looks pretty good to me and from what I can tell Lars addressed
> all of the issues raised by Stas (a lot were referring to __getStatic
> anyway).
> http://lars.schokokeks.org/php/static-classes-002.diff
> Since this one would imo bring a nice addition in regards to handling
> consts as a better choice than defines I'd like to see it in. Also
> only this one seems ready. The __getStatic() indeed seems much more
> complicated as outlined by Stas in detail.

Ok, I guess I totally missed the "static class" patch.
@Lars: Could you open a new thread and start a discussion about that  
patch there?

>

>> In this light the dl() change by Marcus (AFAIK this was planned and
>> done for 6.0 and not 5.3) and the windows PCRE change by Andi seem
>> potential candidates for regressions, issues and maybe should have
>> been discussed beforehand. Just using these two has an example since
>> they are the two last commits I marked as potential issues.
>
> Postponing changes that prevent SEGVs for the sake of versioning and
> that cannot easily be done later because they require c level api
> changes is a no go for me.


Well we for some reason scheduled the change for dl() PHP 6.0 ("enable  
dl() only when a SAPI layer registers it explicitly"). I do not  
remember why we did not schedule it for the next PHP release (maybe we  
just thought PH 6.0 would be the next release) or if we did not want  
to make this change in 5.x. I just noted it as a commit that probably  
should have been mentioned on the list before being committed.

regards,
Lukas Kahwe Smith
mls@pooteeweet.org