[PHP-DEV] RE: Improved zend_string API

2015-06-29 Thread François Laupretre
Hi, OK, I didn’t understand Dmitry’s comment on ‘aggressively’ using inline functions but, thanks to your comment, I understand now. It has nothing to do with performance. Using a function for -val is useless, I agree, as it is read-only be nature. But using 2 functions for len provides

RE: [PHP-DEV] Headsup: PHP7 feature freeze

2015-06-29 Thread Anatol Belski
Hi Yasuo, -Original Message- From: yohg...@gmail.com [mailto:yohg...@gmail.com] On Behalf Of Yasuo Ohgaki Sent: Friday, June 26, 2015 1:58 PM To: Hannes Magnusson Cc: Kalle Sommer Nielsen; Internals; Anatoliy Belsky; Dmitry Stogov; Nikita Popov; Ferenc Kovacs; Xinchen Hui Subject:

[PHP-DEV] Re: Improved zend_string API

2015-06-29 Thread Dmitry Stogov
Committed except of ZVAL_STR_INC/DEC_LEN() and some other unused macros. Thanks. Dmitry. On Mon, Jun 29, 2015 at 4:23 PM, François Laupretre franc...@php.net wrote: Hi, OK, I didn’t understand Dmitry’s comment on ‘aggressively’ using inline functions but, thanks to your comment, I

[PHP-DEV] RE: [PHP-CVS] com php-src: Fix bug #69957 (Different ways of handling div/mod by zero): NEWS Zend/tests/bug69957.phpt Zend/zend_compile.c Zend/zend_operators.c Zend/zend_vm_def.h Zend/zend_v

2015-06-29 Thread Anatol Belski
Hi Bob, This commit and 79b1832dd5086cd9b4a2e778a62f1743b548726c was reverted, please check also the comments in the ticket. Bob, please do more discussion of such cases with Dmitry and Hui before. Especially the patches that affect very basic functionality should not be urged to be pushed

Re: [PHP-DEV] Bundled GD library

2015-06-29 Thread Christoph Becker
Pierre Joye wrote: Hi, On Jun 29, 2015 3:34 AM, Christoph Becker cmbecke...@gmx.de wrote: Hi! I've just learned from ext/gd/libgd/README that the bundled GD library is actually a fork of libgd, which was quite a surprise to me as I thought gd is handled similar to pcre and sqlite3.

Re: [PHP-DEV] Fix division by zero to throw exception (round 2)

2015-06-29 Thread Bob Weinand
I would like to bring this topic back up, as there were users confused with it and it's absolutely not consistent what we have now. See also https://bugs.php.net/bug.php?id=69957 https://bugs.php.net/bug.php?id=69957 (As I thought it was non-intentional, I went ahead and fixed it, was reverted

Re: [PHP-DEV] Fix division by zero to throw exception (round 2)

2015-06-29 Thread Andrea Faulds
Hi Bob, On 29 Jun 2015, at 16:54, Bob Weinand bobw...@hotmail.com wrote: I would like to bring this topic back up, as there were users confused with it and it's absolutely not consistent what we have now. See also https://bugs.php.net/bug.php?id=69957 (As I thought it was

Re: [PHP-DEV] Fix division by zero to throw exception (round 2)

2015-06-29 Thread Bob Weinand
Hey Andrea, Am 29.06.2015 um 18:49 schrieb Andrea Faulds a...@ajf.me: Hi Bob, On 29 Jun 2015, at 16:54, Bob Weinand bobw...@hotmail.com wrote: I would like to bring this topic back up, as there were users confused with it and it's absolutely not consistent what we have now. See also

Re: [PHP-DEV] Fix division by zero to throw exception (round 2)

2015-06-29 Thread Nikita Popov
On Mon, Jun 29, 2015 at 5:54 PM, Bob Weinand bobw...@hotmail.com wrote: I would like to bring this topic back up, as there were users confused with it and it's absolutely not consistent what we have now. See also https://bugs.php.net/bug.php?id=69957 (As I thought it was non-intentional, I

Re: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Bob Weinand
Am 29.06.2015 um 15:46 schrieb Dmitry Stogov dmi...@zend.com: Committed except of ZVAL_STR_INC/DEC_LEN() and some other unused macros. Thanks. Dmitry. On Mon, Jun 29, 2015 at 4:23 PM, François Laupretre franc...@php.net wrote: Hi, OK, I didn’t understand Dmitry’s comment on

Re: [PHP-DEV] Fix division by zero to throw exception (round 2)

2015-06-29 Thread Andrea Faulds
Hi again, On 29 Jun 2015, at 18:02, Bob Weinand bobw...@hotmail.com wrote: Yes, it generally makes sense... Then I have other questions: - Why do we then still have a Warning? Either we have well-defined behavior, or we throw an exception. Well-defined behavior *plus* a warning is IMO

[PHP-DEV] RE: Improved zend_string API

2015-06-29 Thread François Laupretre
Hi Dmitry, I just commited some additional changes, following your suggestions : - Renamed every ‘zend_string_xxx()’ functions to ‘ZSTR_’-prefixed uppercase names, - Defined compatibility macros for old names - Changed the rest of the code to use new ZSTR_ names - A special case for

RE: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Anatol Belski
Hi Bob, -Original Message- From: Bob Weinand [mailto:bobw...@hotmail.com] Sent: Monday, June 29, 2015 6:44 PM To: Dmitry Stogov Cc: francois; Anatol Belski; Anatol Belski; PHP Internals Subject: Re: [PHP-DEV] Improved zend_string API Hey, looks like I'm a bit late to the party,

Re: [PHP-DEV] Headsup: PHP7 feature freeze

2015-06-29 Thread Jakub Kubíček
2015-06-28 22:19 GMT+02:00 Anatol Belski anatol@belski.net: What is the benefit changing it? XHTML is a standard which is alive. That's on every person's opinion. XHTML is currently not a standard, XHTML is dead. But, there's for sure some code based on parsing the phpinfo() output, since

Re: [PHP-DEV] Headsup: PHP7 feature freeze

2015-06-29 Thread Yasuo Ohgaki
Hi Anatol, On Mon, Jun 29, 2015 at 10:19 PM, Anatol Belski anatol@belski.net wrote: -Original Message- From: yohg...@gmail.com [mailto:yohg...@gmail.com] On Behalf Of Yasuo Ohgaki Sent: Friday, June 26, 2015 1:58 PM To: Hannes Magnusson Cc: Kalle Sommer Nielsen;

Re: [PHP-DEV] Move to Fast ZPP?

2015-06-29 Thread Dmitry Stogov
On Sat, Jun 27, 2015 at 12:36 AM, Matt Wilmas php_li...@realplain.com wrote: Hi Dmitry, all, - Original Message - From: Dmitry Stogov Sent: Thursday, June 25, 2015 On Wed, Jun 24, 2015 at 1:35 PM, Matt Wilmas php_li...@realplain.com wrote: Hi Dmitry, all, - Original

RE: [PHP-DEV] Allow exceptions in __toString()

2015-06-29 Thread Anatol Belski
Hi Nikita, -Original Message- From: Nikita Popov [mailto:nikita@gmail.com] Sent: Thursday, June 25, 2015 2:53 PM To: PHP internals; Dmitry Stogov Subject: [PHP-DEV] Allow exceptions in __toString() Hi internals! Currently it's not possible to throw an exception from

Re: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Aaron Piotrowski
On Jun 29, 2015, at 7:37 PM, Bob Weinand bobw...@hotmail.com wrote: Hey Anatol, Am 29.06.2015 um 22:41 schrieb Anatol Belski anatol@belski.net: Hi Bob, -Original Message- From: Bob Weinand [mailto:bobw...@hotmail.com] Sent: Monday, June 29, 2015 6:44 PM To: Dmitry

Re: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Bob Weinand
Hey Dmitry, Am 29.06.2015 um 15:46 schrieb Dmitry Stogov dmi...@zend.com: Committed except of ZVAL_STR_INC/DEC_LEN() and some other unused macros. Thanks. Dmitry. I see you changed the ZSTR_* macros to be pure aliases of the equivalent structure members. May I now honestly ask where the

RE: [PHP-DEV] Improved zend_string API

2015-06-29 Thread François Laupretre
De : Aaron Piotrowski [mailto:aa...@icicle.io] Envoyé : mardi 30 juin 2015 04:16 À : Bob Weinand; Anatol Belski Cc : internals@lists.php.net Objet : Re: [PHP-DEV] Improved zend_string API On Jun 29, 2015, at 7:37 PM, Bob Weinand bobw...@hotmail.com wrote: Hey Anatol, Am

Re: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Bob Weinand
Hey Anatol, Am 29.06.2015 um 22:41 schrieb Anatol Belski anatol@belski.net: Hi Bob, -Original Message- From: Bob Weinand [mailto:bobw...@hotmail.com] Sent: Monday, June 29, 2015 6:44 PM To: Dmitry Stogov Cc: francois; Anatol Belski; Anatol Belski; PHP Internals Subject:

RE: [PHP-DEV] Improved zend_string API

2015-06-29 Thread François Laupretre
Hi, -Message d'origine- De : Bob Weinand [mailto:bobw...@hotmail.com] Hey, looks like I'm a bit late to the party, but I'd like to object against the accessor macros on zend_string. What does it help to replace very nice -val, -len etc. with macros? If you find it 'very nice',

RE: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Pierre Joye
On Jun 30, 2015 9:38 AM, François Laupretre franc...@php.net wrote: Hi, -Message d'origine- De : Bob Weinand [mailto:bobw...@hotmail.com] Hey, looks like I'm a bit late to the party, but I'd like to object against the accessor macros on zend_string. What does it help to

RE: [PHP-DEV] RE: Improved zend_string API

2015-06-29 Thread François Laupretre
Hi, De : François Laupretre [mailto:franc...@php.net] Envoyé : mardi 30 juin 2015 03:08 À : 'Dmitry Stogov' Cc : 'Anatol Belski'; 'PHP Internals' Objet : [PHP-DEV] RE: Improved zend_string API Hi Dmitry, I just commited some additional changes, following your suggestions : - Renamed

RE: [PHP-DEV] Improved zend_string API

2015-06-29 Thread François Laupretre
De : Bob Weinand [mailto:bobw...@hotmail.com] I see you changed the ZSTR_* macros to be pure aliases of the equivalent structure members. May I now honestly ask where the benefit of those macros is? ZSTR_VAL(str) instead of str-val ZSTR_H(str) instead of str-h ZSTR_LEN(str) instead of

Re: [PHP-DEV] Headsup: PHP7 feature freeze

2015-06-29 Thread Levi Morrison
On Mon, Jun 29, 2015 at 11:31 PM, Levi Morrison le...@php.net wrote: On Mon, Jun 29, 2015 at 3:47 PM, Jakub Kubíček kelerest...@gmail.com wrote: 2015-06-28 22:19 GMT+02:00 Anatol Belski anatol@belski.net: What is the benefit changing it? XHTML is a standard which is alive. That's on every

[PHP-DEV] VCS Account Request: trowski

2015-06-29 Thread Aaron Piotrowski
Quickly applying bug fixes. -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php

Re: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Bob Weinand
Am 30.06.2015 um 04:37 schrieb François Laupretre franc...@php.net: Hi, -Message d'origine- De : Bob Weinand [mailto:bobw...@hotmail.com] Hey, looks like I'm a bit late to the party, but I'd like to object against the accessor macros on zend_string. What does it help to

Re: [PHP-DEV] Improved zend_string API

2015-06-29 Thread Bob Weinand
Am 30.06.2015 um 05:44 schrieb François Laupretre franc...@php.net: De : Bob Weinand [mailto:bobw...@hotmail.com] I see you changed the ZSTR_* macros to be pure aliases of the equivalent structure members. May I now honestly ask where the benefit of those macros is? ZSTR_VAL(str)

Re: [PHP-DEV] Headsup: PHP7 feature freeze

2015-06-29 Thread Levi Morrison
On Mon, Jun 29, 2015 at 3:47 PM, Jakub Kubíček kelerest...@gmail.com wrote: 2015-06-28 22:19 GMT+02:00 Anatol Belski anatol@belski.net: What is the benefit changing it? XHTML is a standard which is alive. That's on every person's opinion. XHTML is currently not a standard, XHTML is dead.