[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-12-19 Thread Xinchen Hui
laruence Mon, 19 Dec 2011 10:05:48 + Revision: http://svn.php.net/viewvc?view=revision&revision=321159 Log: Codes standard & Remove useless empty lines Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-12-05 Thread Pierrick Charron
pierrick Tue, 06 Dec 2011 06:44:22 + Revision: http://svn.php.net/viewvc?view=revision&revision=320485 Log: Coding standard Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified: php/php

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-12-05 Thread Xinchen Hui
laruence Mon, 05 Dec 2011 12:33:56 + Revision: http://svn.php.net/viewvc?view=revision&revision=320425 Log: Fix typo Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified: php/php-src/br

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Pierre Joye
On Fri, Nov 18, 2011 at 4:03 PM, Stefan Marr wrote: > > On 18 Nov 2011, at 15:36, Felipe Pena wrote: >> 'Be generous with whitespace' is the opposite of what you want to do. >> I.e. generous != stingy. :P > > I was obviously to imprecise in my request: I do not want to sound harsh but discussions

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Stefan Marr
On 18 Nov 2011, at 15:36, Felipe Pena wrote: > 'Be generous with whitespace' is the opposite of what you want to do. > I.e. generous != stingy. :P I was obviously to imprecise in my request: Index: Zend/zend_compile.c === --- Zend/z

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Felipe Pena
> Then at least follow what 3. states as the first (and deducted from that) > most important statement: > >   "Be generous with whitespace" > > Please, make it look like a table. > That is not against the coding standards.  (Which IMHO should leave room for > exceptions when it comes to readabili

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Stefan Marr
Hi: On 18 Nov 2011, at 15:16, Felipe Pena wrote: > Hi, > > 2011/11/18 Stefan Marr : >> Hi Felipe: >> >> On 18 Nov 2011, at 14:56, Felipe Pena wrote: >> >>> felipe Fri, 18 Nov 2011 13:56:41 + >>> >>> Revision: http://svn.php.net/viewvc?view=revision&revisi

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Felipe Pena
Hi, 2011/11/18 Stefan Marr : > Hi Felipe: > > On 18 Nov 2011, at 14:56, Felipe Pena wrote: > >> felipe                                   Fri, 18 Nov 2011 13:56:41 + >> >> Revision: http://svn.php.net/viewvc?view=revision&revision=319487 >> >> -     else if (!strncmp(mname, ZEND_DESTRUCTOR_FUNC

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Pierre Joye
On Fri, Nov 18, 2011 at 3:07 PM, Stefan Marr wrote: > Would it be a problem to revert that change, or at least 'table' it again? http://svn.php.net/viewvc/php/php-src/trunk/CODING_STANDARDS?revision=296679&view=markup Nothing to discuss, love, hate, etc., just apply them :-) Cheers, -- Pierre

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Stefan Marr
Hi Felipe: On 18 Nov 2011, at 14:56, Felipe Pena wrote: > felipe Fri, 18 Nov 2011 13:56:41 + > > Revision: http://svn.php.net/viewvc?view=revision&revision=319487 > > - else if (!strncmp(mname, ZEND_DESTRUCTOR_FUNC_NAME, mname_len)) { > ce->destr

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-18 Thread Felipe Pena
felipe Fri, 18 Nov 2011 13:56:41 + Revision: http://svn.php.net/viewvc?view=revision&revision=319487 Log: - CS Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified: php/php-src/branch

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-17 Thread Felipe Pena
felipe Thu, 17 Nov 2011 21:07:52 + Revision: http://svn.php.net/viewvc?view=revision&revision=319422 Log: - Fix build Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified: php/php-src

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-04 Thread Stefan Marr
gron Sat, 05 Nov 2011 02:05:28 + Revision: http://svn.php.net/viewvc?view=revision&revision=318794 Log: Fixed inconsistent whitespace. # Belongs to svn rev. 318793. Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-03 Thread Antony Dovgal
On 11/03/2011 01:21 PM, Stefan Marr wrote: Hi Antony: On 02 Nov 2011, at 22:05, Antony Dovgal wrote: tony2001 Wed, 02 Nov 2011 21:05:36 + Revision: http://svn.php.net/viewvc?view=revision&revision=318705 Log: initialize variable and fix segfaulting tes

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-03 Thread Stefan Marr
Hi Antony: On 02 Nov 2011, at 22:05, Antony Dovgal wrote: > tony2001 Wed, 02 Nov 2011 21:05:36 + > > Revision: http://svn.php.net/viewvc?view=revision&revision=318705 > > Log: > initialize variable and fix segfaulting tests Thanks for catching this one and t

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-03 Thread Antony Dovgal
tony2001 Thu, 03 Nov 2011 08:39:12 + Revision: http://svn.php.net/viewvc?view=revision&revision=318723 Log: use jmp_addr only when it's initialized Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_comp

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-02 Thread Antony Dovgal
tony2001 Wed, 02 Nov 2011 21:12:13 + Revision: http://svn.php.net/viewvc?view=revision&revision=318706 Log: fix folding, ws and cs Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified:

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-11-02 Thread Antony Dovgal
tony2001 Wed, 02 Nov 2011 21:05:36 + Revision: http://svn.php.net/viewvc?view=revision&revision=318705 Log: initialize variable and fix segfaulting tests Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zen

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-10-08 Thread Stefan Marr
gron Sat, 08 Oct 2011 23:47:16 + Revision: http://svn.php.net/viewvc?view=revision&revision=317930 Log: Use ZEND_ constants instead of plain strings for names of magic methods. # No functional changes. Changed paths: U php/php-src/branches/PHP_5_4/Ze

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-10-06 Thread Xinchen Hui
laruence Fri, 07 Oct 2011 04:14:31 + Revision: http://svn.php.net/viewvc?view=revision&revision=317849 Log: Remove unecessary codes Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified:

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-09-13 Thread Derick Rethans
derick Tue, 13 Sep 2011 15:17:11 + Revision: http://svn.php.net/viewvc?view=revision&revision=316654 Log: - Fixed compiler warnings (patch by Kalle Sommer Nielsen) Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/tr

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-17 Thread Johannes Schlüter
On Tue, 2011-08-16 at 12:37 -0700, Stas Malyshev wrote: > Hi! > > On 8/16/11 6:19 AM, Johannes Schlüter wrote: > > It saves one character of typing. > > For me, it's the reason enough to prohibit it altogether :) Writing > weird code to save one character of typing is the worst habit we could >

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Stas Malyshev
Hi! On 8/16/11 6:19 AM, Johannes Schlüter wrote: It saves one character of typing. For me, it's the reason enough to prohibit it altogether :) Writing weird code to save one character of typing is the worst habit we could promote. But I digress since the idea of this patch was obviously not

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Hannes Magnusson
2011/8/16 Johannes Schlüter : > On Tue, 2011-08-16 at 13:44 +0200, Hannes Magnusson wrote: >> "use foobar;" has no effect, at all, and you will get an warning >> message telling you that. > > It has an effect. In > > namespace strict { class C {} } > namespace foo { use strict; new strict\C; } > >

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Johannes Schlüter
On Tue, 2011-08-16 at 13:44 +0200, Hannes Magnusson wrote: > "use foobar;" has no effect, at all, and you will get an warning > message telling you that. It has an effect. In namespace strict { class C {} } namespace foo { use strict; new strict\C; } It saves one character of typing. Without the

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Hannes Magnusson
On Tue, Aug 16, 2011 at 13:39, Stefan Marr wrote: > > On 16 Aug 2011, at 13:23, Hannes Magnusson wrote: > >> On Tue, Aug 16, 2011 at 13:05, Stefan Marr wrote: >>> Hi Hannes: >>> >>> On 16 Aug 2011, at 12:46, Hannes Magnusson wrote: >>> +             if (!strcmp(Z_STRVAL_P(name), "strict")) {

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Stefan Marr
On 16 Aug 2011, at 13:23, Hannes Magnusson wrote: > On Tue, Aug 16, 2011 at 13:05, Stefan Marr wrote: >> Hi Hannes: >> >> On 16 Aug 2011, at 12:46, Hannes Magnusson wrote: >> >>> + if (!strcmp(Z_STRVAL_P(name), "strict")) { >>> + zend_error(E_COMPILE_ERROR, "You

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Hannes Magnusson
On Tue, Aug 16, 2011 at 13:05, Stefan Marr wrote: > Hi Hannes: > > On 16 Aug 2011, at 12:46, Hannes Magnusson wrote: > >> +             if (!strcmp(Z_STRVAL_P(name), "strict")) { >> +                     zend_error(E_COMPILE_ERROR, "You seem to be trying to >> use a different language..."); >> +

Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Stefan Marr
Hi Hannes: On 16 Aug 2011, at 12:46, Hannes Magnusson wrote: > + if (!strcmp(Z_STRVAL_P(name), "strict")) { > + zend_error(E_COMPILE_ERROR, "You seem to be trying to > use a different language..."); > + } Could we change that error to be a bit more exp

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-16 Thread Hannes Magnusson
bjoriTue, 16 Aug 2011 10:46:49 + Revision: http://svn.php.net/viewvc?view=revision&revision=315002 Log: Improve errormessage on wrong use statements Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-08-15 Thread Stefan Marr
gron Mon, 15 Aug 2011 08:29:48 + Revision: http://svn.php.net/viewvc?view=revision&revision=314921 Log: Addendum for patch to bug #55214: Class name was freed before method literal referring to it. # Thanks to Felipe for catching this. # The fix duplicates

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-07-31 Thread Felipe Pena
felipe Sun, 31 Jul 2011 12:27:47 + Revision: http://svn.php.net/viewvc?view=revision&revision=313991 Log: - Fixed bug #55326 (Trait method copying misindentifies serialization callbacks) patch by: Gustavo Lopes (cataphract@php) Bug: https://bugs.php.net/55

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-07-23 Thread Stefan Marr
gron Sat, 23 Jul 2011 18:12:31 + Revision: http://svn.php.net/viewvc?view=revision&revision=313638 Log: Fixed incorrect initialization when copied op_array.vars was NULL (TRAITS) Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-07-11 Thread Dmitry Stogov
dmitry Mon, 11 Jul 2011 07:10:30 + Revision: http://svn.php.net/viewvc?view=revision&revision=313125 Log: Fixed bug #55150 (php -a segfaults) Bug: https://bugs.php.net/55150 (Assigned) php -a segfaults Changed paths: U php/php-src/branches/PHP_5

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-07-09 Thread Felipe Pena
felipe Sat, 09 Jul 2011 12:06:11 + Revision: http://svn.php.net/viewvc?view=revision&revision=313085 Log: - Minor optimization and clean up Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-07-08 Thread Felipe Pena
felipe Fri, 08 Jul 2011 19:52:21 + Revision: http://svn.php.net/viewvc?view=revision&revision=313078 Log: - CS and WS Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified: php/php-src

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-06-19 Thread Felipe Pena
felipe Mon, 20 Jun 2011 02:04:13 + Revision: http://svn.php.net/viewvc?view=revision&revision=312322 Log: - Optimize property finding by zend_hash_quick_find Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Ze

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-06-19 Thread Felipe Pena
felipe Mon, 20 Jun 2011 01:36:23 + Revision: http://svn.php.net/viewvc?view=revision&revision=312321 Log: - Simplify function code Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-06-19 Thread Felipe Pena
felipe Mon, 20 Jun 2011 01:21:47 + Revision: http://svn.php.net/viewvc?view=revision&revision=312320 Log: - Avoid re-hash here too Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-06-19 Thread Felipe Pena
felipe Sun, 19 Jun 2011 20:16:08 + Revision: http://svn.php.net/viewvc?view=revision&revision=312303 Log: - Removed unnecessary TSRMLS_FETCH Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c

[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/zend_compile.c

2011-05-17 Thread Pierre Joye
pajoye Tue, 17 May 2011 10:39:18 + Revision: http://svn.php.net/viewvc?view=revision&revision=311123 Log: - WS Changed paths: U php/php-src/branches/PHP_5_4/Zend/zend_compile.c U php/php-src/trunk/Zend/zend_compile.c Modified: php/php-src/branch