Re: [PATCH] Property Getters/Setters (v2.4) for review
| From: | John LeSueur | Date: | Fri, 03 Feb 2012 13:54:12 +0000 |
| Subject: | Re: [PATCH] Property Getters/Setters (v2.4) for review | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-57659@lists.php.net to get a copy of this message | ||
Because Zend/zend_language_scanner.c is a generated file, you could exclude
it from your diff, and end up with a much smaller patch. Smaller patches
are more appealing :)
On Fri, Feb 3, 2012 at 6:47 AM, Clint M Priest <cpriest@zerocue.com> wrote:
> The property accessor functionality is done and is detailed here:
> https://wiki.php.net/rfc/propertygetsetsyntax-as-implemented
>
> Core Patch:
> http://www.clintpriest.com/patches/php-core/accessors/2.4.diff
>
> The core patch encompasses the entire original RFC and the as-implemented
> includes implementation details and edge cases not defined by the original
> RFC.
>
> Most changes are in zend_compile, zend_object_handlers and
> zend_language_scanner.
>
> Thanks,
>
> -Clint
>