Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:78972
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.217.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAFMT4NpJxuuqc-wTr1aE4K+wZBT8+04j+9-SsY+nhUMU83fDUg@mail.gmail.com>
References: <CAFMT4NpJxuuqc-wTr1aE4K+wZBT8+04j+9-SsY+nhUMU83fDUg@mail.gmail.com>
Date: Wed, 19 Nov 2014 00:26:51 +0100
Message-ID: <CAESvBjmg0VNnTBc=0imkRfk0BK_UUXJ=zYQg47fpQ9Lrbu=-cw@mail.gmail.com>
To: Levi Morrison <levim@php.net>
Cc: internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=001a11c33fcc3a4c0505082a6ffd
Subject: Re: [PHP-DEV] [RFC] Remove PHP 4 Constructors
From: julien.breux@gmail.com (Julien Breux)

--001a11c33fcc3a4c0505082a6ffd
Content-Type: text/plain; charset=UTF-8

I think that it is great time to end to PHP 4 constructors system for PHP 7.

IMO, It's a good RFC.


On Wed, Nov 19, 2014 at 12:11 AM, Levi Morrison <levim@php.net> wrote:

> Dear Internals,
>
> I am proposing an RFC[1] to remove PHP 4 constructors in PHP 7. If
> accepted, methods with the same name as their defining class will no
> longer be recognized as constructors. As noted in the RFC, there are
> already many situations where we do not recognize these methods as
> constructors, such as in namespaces and traits and when `function
> __construct` is also present.
>
> Andrea Faulds has kindly written a utility that identifies when a PHP
> 4 constructor is defined[2]. It does not automatically change the code
> for liability reasons. The utility PHPMD[3] can also detect this but
> has a false positive when `__construct` is also defined.
>
> Cheers,
> Levi Morrison
>
>
>   [1]: https://wiki.php.net/rfc/remove_php4_constructors
>   [2]: https://github.com/TazeTSchnitzel/PHP4_Constructor_Finder
>   [3]:
> http://phpmd.org/rules/naming.html#constructorwithnameasenclosingclass
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

--001a11c33fcc3a4c0505082a6ffd--