Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:85795
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.213.172 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CB.90.41712.5B6CB255@pb1.pair.com>
References: <CB.90.41712.5B6CB255@pb1.pair.com>
Date: Mon, 13 Apr 2015 21:39:02 -0300
Message-ID: <CAOsHV+vF2aKopRrQeYDoHtHC-mJoahCt7sh4=7jtJn_4xTj5sA@mail.gmail.com>
To: Johannes Ott <mail@deroetzi.de>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV] New RFC draft "static class constructor"
From: marcio.web2@gmail.com (Marcio Almada)

Hi,

2015-04-13 10:37 GMT-03:00 Johannes Ott <mail@deroetzi.de>:
> Hi,
>
> finally I managed to do my first RFC draft.
>
> https://wiki.php.net/rfc/static_class_constructor
>
> I hope I have done everything correct so far and I'm looking forward to
> your feedback on it.
>
> As I already mentioned in the prediscussion thread here:
>
> For being my first change to the PHP core, I would be very happy, if
> someone of the "old stager" would help me with the implementation of
> this RFC if it is accepted.
>
> Regards
>
> --
> DerOetzi
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>

I know this is still a draft, but conceptually I don't like this
feature. I always avoid static
state on classes as much as possible and I think this should be
avoided. But this is my
personal opinion. Some suggestions:

In case the RFC passes, I'd prefer to have a `static{ ... }` block
instead of yet
another static __ method. The reason is that this kind of construct is
already known
on other languages as `static{...}` and that feels more organized IMMO.

In case you decide to proceed with the magic method approach, I'd
prefer the more
verbose `__staticConstruct` name because it seems clearer.

Thanks,
M=C3=A1rcio