Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:71980
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain pthreads.org from 74.125.82.53 cause and error)
Message-ID: <52EE1072.8050604@pthreads.org>
Date: Sun, 02 Feb 2014 09:31:30 +0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7
MIME-Version: 1.0
To: =?UTF-8?B?Sm9oYW5uZXMgU2NobMO8dGVy?= <johannes@schlueters.de>, 
 PHP internals <internals@lists.php.net>
References: <98.E0.35265.E17FBE25@pb1.pair.com>  <1391203805.2941.173.camel@guybrush>  <BF.14.54292.9A62CE25@pb1.pair.com> <1391210014.2941.224.camel@guybrush>
In-Reply-To: <1391210014.2941.224.camel@guybrush>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-DEV] [VOTE] Automatic Property Initialization
From: pthreads@pthreads.org (Joe Watkins)

Morning internalz,

Some clarity ...

 > Johannes:
 > it there seems to be a design flaw around inheritance with code like
 > this:
 >     class Base { public $p; function __construct($this->p = 42); }
 >     Class E extends Base { function __construct($this->p = 23) { 
parent::__construct(); }
 >     new E;

Assuming you meant to declare Base::__construct abstract:

L1946 zend_compile.c:

 > zend_error(E_COMPILE_ERROR, "Automatic property initialization can 
not be used in an abstract method");

 > leading to probably unexpected results ... also
 >     class C { public function m($foo->prop){}}

L1933 zend_compile.c:

 > zend_error(E_COMPILE_ERROR, "Property name can only be prefixed by 
$this->");

Having reviewed the discussion, and played with the patch, I see no 
genuine objection.

Cheers
Joe