Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Lars Strojny
Hi, Am Sonntag, den 10.02.2008, 21:11 -0800 schrieb Stanislav Malyshev: [...] Actually, just as __FILE__, __LINE__, __CLASS__ etc. dirname(__FILE__) can be a parser constant. And if somebody produces a good name and a patch for it, I see no reason to reject it. Does anybody? So here we are.

Re: [PHP-DEV] w32api extension

2008-02-11 Thread Pierre Joye
On Feb 11, 2008 8:36 PM, Justin Martin [EMAIL PROTECTED] wrote: Pierre Joye wrote: Hi Justin, ... It has been moved out of php-src because the extension was not maintained. It is in PECL's cvs but has no release (still not maintained): http://cvs.php.net/viewvc.cgi/pecl/w32api/

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Stanislav Malyshev
and it should be good. Stan, looks ok for you? Done: http://lars.schokokeks.org/php/dir-constant-6.diff I'm not sure what happens if zend_dirname returns 0, also what happens if this constant is used in eval(), stream of anything else with non-canonical filename? Except that, looks ok.

[PHP-DEV] Re: [RFC] Conditional INI support

2008-02-11 Thread Edward Z. Yang
Why reinvent the wheel: just let PHP (with the `-n` flag) output, to stdout, the ini file? :o) -- Edward Z. YangGnuPG: 0x869C48DA HTML Purifier http://htmlpurifier.org Anti-XSS Filter [[ 3FA8 E9A9 7385 B691 A6FC B3CB A933 BE7D 869C 48DA ]] -- PHP Internals - PHP

Re: [PHP-DEV] w32api extension

2008-02-11 Thread Justin Martin
Pierre Joye wrote: Hi Justin, ... It has been moved out of php-src because the extension was not maintained. It is in PECL's cvs but has no release (still not maintained): http://cvs.php.net/viewvc.cgi/pecl/w32api/ FFI is much more powerful yes. But it has been developed independently of

Re: [PHP-DEV] Updates to the QA pages

2008-02-11 Thread Philip Olson
Rob Nicholson and I have made a set updates the QA pages, mainly based on things that we've learned over the past year of test case writing - we had a lot of input from Raghu and Marcus on this too of course. The updated pages are here for review: http://zapt.info/phpt/write- test.php.

[PHP-DEV] substr substr_compare fix

2008-02-11 Thread Etienne Kneuss
Hi, please consider the following patches [1,2] that are supposed to fix 1) substr in 5_3 2) substr_compare in 5_3 and HEAD It basically makes them not so strict about the length argument. Regards [1] http://patches.colder.ch/php-src/string_substr_5_3.patch?markup [2]

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Johannes Schlüter
On Mon, 2008-02-11 at 17:51 +0100, Pierre Joye wrote: Yes that name is fine, bunt wondering: When/Where do you get this information and where from? What's the impact for build snapshots/releases? Afaik the scripts for that work on cvs exports where Good point, I did not check the CVS doc

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Lars Strojny
Hi, Am Montag, den 11.02.2008, 17:58 +0100 schrieb Pierre Joye: [...] One more thing: const size_t filename_len = strlen(filename); and it should be good. Stan, looks ok for you? Done: http://lars.schokokeks.org/php/dir-constant-6.diff cu, Lars signature.asc Description: Dies ist ein

Re: [PHP-DEV] CVS Account Request: nexuslite

2008-02-11 Thread Antony Dovgal
On 11.02.2008 17:00, Marcus Kelly wrote: Maintaining an official, bundled PHP extension: dBase for starters may also pick up a couple other abandoned extensions if time permits. The common practice is to start with patches, CVS account will follow later if the patches are ok. -- Wbr,

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Lars Strojny
Hi, Am Montag, den 11.02.2008, 13:25 -0800 schrieb Stanislav Malyshev: [...] I'm not sure what happens if zend_dirname returns 0, also what happens if this constant is used in eval(), stream of anything else with non-canonical filename? Except that, looks ok. Some tests won't hurt though :)

RE: [PHP-DEV] [RFC] Conditional INI support

2008-02-11 Thread Andi Gutmans
Hi Marcus, In general I think conditional INI support can benefit many of our users. We just need to make sure that we cover the most common needs and also that we keep it very basic and simple so that we don't boil the ocean and maintain two languages. So it's a balance we need to meet. It

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Lars Strojny
Hi Pierre, Am Montag, den 11.02.2008, 17:42 +0100 schrieb Pierre Joye: [...] Looks better :) Thank you! I did not test it but it looks good. Did you run the tests suite? + dirname = estrndup(filename, strlen(filename)); + zend_dirname(dirname, strlen(dirname)); One strlen can be

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Pierre Joye
On Feb 11, 2008 5:40 PM, Johannes Schlüter [EMAIL PROTECTED] wrote: Pierre, On Mon, 2008-02-11 at 15:42 +0100, Pierre Joye wrote: On Feb 11, 2008 3:20 PM, Pierre Joye [EMAIL PROTECTED] wrote: On Feb 9, 2008 2:39 PM, Marcus Boerger [EMAIL PROTECTED] wrote: Hello Pierre, yeah nice

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Johannes Schlüter
Pierre, On Mon, 2008-02-11 at 15:42 +0100, Pierre Joye wrote: On Feb 11, 2008 3:20 PM, Pierre Joye [EMAIL PROTECTED] wrote: On Feb 9, 2008 2:39 PM, Marcus Boerger [EMAIL PROTECTED] wrote: Hello Pierre, yeah nice idea. I am wondering if you might want to add the cvs tag, too?

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Lars Strojny
Hi, Am Montag, den 11.02.2008, 12:51 +0100 schrieb Lars Strojny: [...] [1] http://lars.schokokeks.org/php/dir-constant-2.diff Updated version. Moved and renamed php_dirname() to zend_dirname() to not use PHP internals in the Zend Engine. Fixed a memory leak and added the TSRM-includes to have

Re: [PHP-DEV] [RFC] Conditional INI support

2008-02-11 Thread Jani Taskinen
On Sat, 2008-02-09 at 15:33 +0100, Marcus Boerger wrote: 3) Add more values to INI parsing, namely: ${php.version} = 50300 ${php.debug} = 0 ${php.zts} = 0 ${php.sapi} = CLI And btw. This also works already: php_sapi = PHP_SAPI Parser checks this kind of constant strings for equialent in

Re: [PHP-DEV] Problems with LSB

2008-02-11 Thread Moritz Bechler
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 My original reply missed the list :| Hi, i've written the following code using Etiennes LSB. But I'm facing some problems. ?php class A { function foo() { echo get_called_class(); } } class B

Re: [PHP-DEV] Problems with LSB

2008-02-11 Thread Sebastian Deutsch
Hello, that's it. Even $b = new B; $b-foo(); // echos B - good works. Thank you! *.sebastian Sebastian Deutsch schrieb: Hello, care... my case is slightly different. I was aware of that problem, but in my case I call B::foo() from the main scope - it behaves right - when I call it

Re: [PHP-DEV] Problems with LSB

2008-02-11 Thread Sebastian Deutsch
Hello, care... my case is slightly different. I was aware of that problem, but in my case I call B::foo() from the main scope - it behaves right - when I call it within the scope of C (same call) it behaves different. This is different as described in the bug. The same call should have the same

Re: [PHP-DEV] Problems with LSB

2008-02-11 Thread Lokrain
Hello, Sebastian This seems to be a known bug http://bugs.php.net/bug.php?id=43408 and in fact already assigned. Fallbacks occur in static/self calls, as static/self resolve to foo and it returns foo as expected. However, when you do a parent::demo() you actually call bar::demo(), which is

Re: [PHP-DEV] magic_quotes and the question of BC

2008-02-11 Thread Jani Taskinen
If those application are written properly, they also have to check for the existance of the functions even with current PHP version since it is possible to disable them altogether. So it shouldn't matter if they're removed for them.. ;) --Jani On Fri, 2008-02-08 at 10:38 -0600, Gregory Beaver

Re: [PHP-DEV] Re: BC break in DirectoryIterator by fix of #44018

2008-02-11 Thread Marcus Boerger
Hello Gregory, hmm, and i thought i was doing it correctly now. So I need to reinvestigate things. Thanks fo rthe heads up. marcus Monday, February 11, 2008, 6:57:09 AM, you wrote: Gregory Beaver wrote: Hi Marcus, FYI, this change:

[PHP-DEV] PHP 4 Bug Summary Report

2008-02-11 Thread internals
PHP 4 Bug Database summary - http://bugs.php.net Num Status Summary (625 total including feature requests) ===[*Compile Issues]== 43389 Open configure ignoring --without-cdb flag

Re: [PHP-DEV] magic_quotes and the question of BC

2008-02-11 Thread Gregory Beaver
Jani Taskinen wrote: If those application are written properly, they also have to check for the existance of the functions even with current PHP version since it is possible to disable them altogether. So it shouldn't matter if they're removed for them.. ;) It's possible to disable any

[PHP-DEV] PHP 5.2.6

2008-02-11 Thread Ilia Alshanetsky
It has been a while since the last PHP 5.2 release and a fair number of fixes has accumulated. I'd like to start on the release process, with RC1 scheduled for 2 weeks from now (Feb, 28th). If you have any pending bug fixes for this branch, please try to get them in before that time.

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Pierre Joye
On Feb 11, 2008 3:20 PM, Pierre Joye [EMAIL PROTECTED] wrote: On Feb 9, 2008 2:39 PM, Marcus Boerger [EMAIL PROTECTED] wrote: Hello Pierre, yeah nice idea. I am wondering if you might want to add the cvs tag, too? E.g.: PHP_TAG = 'PHP_5_3' I'll commit the patch later today, I think

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Pierre Joye
On Feb 9, 2008 2:39 PM, Marcus Boerger [EMAIL PROTECTED] wrote: Hello Pierre, yeah nice idea. I am wondering if you might want to add the cvs tag, too? E.g.: PHP_TAG = 'PHP_5_3' I'll commit the patch later today, I think PHP_BRANCHE sounds better than PHP_TAG, or PHP_CVS_TAG (but if we move

[PHP-DEV] Problems with LSB

2008-02-11 Thread Etienne Kneuss
Hello, your foo function is not defined as static, hence no static call is done. Strict Standards: Non-static method A::foo() should not be called statically, assuming $this from incompatible context ... Define your method as static and it should work just fine. Regards On Feb 11, 2008

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Pierre Joye
On Feb 11, 2008 4:25 PM, Lars Strojny [EMAIL PROTECTED] wrote: Hi, Am Montag, den 11.02.2008, 12:51 +0100 schrieb Lars Strojny: [...] [1] http://lars.schokokeks.org/php/dir-constant-2.diff Updated version. Moved and renamed php_dirname() to zend_dirname() to not use PHP internals in the

Re: [PHP-DEV] _REQUEST and variable_order

2008-02-11 Thread Derick Rethans
On Wed, 6 Feb 2008, Stanislav Malyshev wrote: So, what do you think on this? Don't care so much about it, as long as the defaults keep the same like they are now. Derick -- Derick Rethans http://derickrethans.nl | http://ezcomponents.org | http://xdebug.org -- PHP Internals - PHP Runtime

Re: [PHP-DEV] [RFC] Conditional INI support

2008-02-11 Thread Derick Rethans
On Sun, 10 Feb 2008, Stanislav Malyshev wrote: 3) Add more values to INI parsing, namely: ${php.version} = 50300 ${php.debug} = 0 ${php.zts} = 0 ${php.sapi} = CLI What these would be used for? The version one is atleast useful for me to load different variants of xdebug without

Re: [PHP-DEV] w32api extension

2008-02-11 Thread Pierre Joye
Hi Justin, On Feb 11, 2008 5:43 AM, Justin Martin [EMAIL PROTECTED] wrote: Hello there, Just wondering if there is any interest in bringing w32api up to speed. The docs say it was moved to PECL as of 5.1.0, however is this not the case. Rumor has it that it was to be replaced by ffi, however

[PHP-DEV] PHP 6 Bug Summary Report

2008-02-11 Thread internals
PHP 6 Bug Database summary - http://bugs.php.net Num Status Summary (59 total including feature requests) ===[*General Issues]== 26771 Suspended register_tick_funtions crash under threaded webservers

Re: [PHP-DEV] magic_quotes and the question of BC

2008-02-11 Thread Tomas Kuliavas
If those application are written properly, they also have to check for the existance of the functions even with current PHP version since it is possible to disable them altogether. So it shouldn't matter if they're removed for them.. ;) Function is standard for PHP4-5. get_magic_quotes_gpc()

Re: [PHP-DEV] [RFC] Conditional INI support

2008-02-11 Thread Jani Taskinen
On Mon, 2008-02-11 at 10:32 +0100, Marcus Boerger wrote: Hello Derick, exactly why I spent the time working on this. The ability to load the correct stuff while developing. The sapi stuff might be a better solution for sapi specific stuff as then you do not need to pass several ini files.

Re: [PHP-DEV] [patch] expose PHP version details as constants

2008-02-11 Thread Pierre Joye
On Feb 11, 2008 5:55 PM, Lars Strojny [EMAIL PROTECTED] wrote: Hi Pierre, Am Montag, den 11.02.2008, 17:42 +0100 schrieb Pierre Joye: [...] Looks better :) Thank you! I did not test it but it looks good. Did you run the tests suite? + dirname = estrndup(filename,