Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:104069
MIME-Version: 1.0
References: <CAL=_i_=KCzMg63OPSGfjnb5m2sjK-5XTuCfokGMBwP-RBFH5Ag@mail.gmail.com>
 <ca713698-78fe-55bc-21df-0f7ce9c99de1@gmx.de>
In-Reply-To: <ca713698-78fe-55bc-21df-0f7ce9c99de1@gmx.de>
Date: Sun, 3 Feb 2019 18:52:09 +0100
Message-ID: <CAOwTYAs-JCtLKQC2gz8QQzWhHviGdL90mOBFY7d8F2uYx80Zkw@mail.gmail.com>
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: PHP internals <internals@lists.php.net>, colder@php.net
Content-Type: multipart/alternative; boundary="000000000000aaec05058101099a"
Subject: Re: RFC Weakrefs
From: krakjoe@gmail.com (Joe Watkins)

--000000000000aaec05058101099a
Content-Type: text/plain; charset="UTF-8"

Afternoon Christoph,

I can't really think of another name ... it's ... a weakref ...

Probably using the PHP namespace makes most sense.

Cheers
Joe

On Sun, 3 Feb 2019 at 18:40, Christoph M. Becker <cmbecker69@gmx.de> wrote:

> On 02.02.2019 at 09:35, Joe Watkins wrote:
>
> > Some time ago I brought this up for discussion, and last night was
> reminded
> > of it's existence, and so this morning rebased and reworked the patch a
> > little based on the feedback I got back then.
> >
> > Since it was long ago, and there's no particular rush, I don't intend to
> > open voting until the current policy adjustment RFC's are resolved, but
> > wanted to give everyone a heads up and ask for any feedback you may have
> at
> > this time.
> >
> > https://wiki.php.net/rfc/weakrefs
>
> Overall I like this.  Thanks!
>
> However, calling this class WeakRef would introduce a potential BC
> break, and would be a particular issue regarding the PHP manual, which
> does not allow to document two classes with exactly the same name.
>
> Even if Etienne would officially cease the further development of
> PECL/Weakref[1], having a class with the same name appears to be
> confusing at best.
>
> Maybe another name should be chosen for the class?  Or maybe we should
> finally introduce the \PHP namespace?
>
> [1] <https://pecl.php.net/package/Weakref>
>
> --
> Christoph M. Becker
>

--000000000000aaec05058101099a--