Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114873
MIME-Version: 1.0
References: <CAF+90c_2TTC=MiZmmAeUC52iiLU8L3cvvyuZoTwamP5VU79uBQ@mail.gmail.com>
 <88132afe-0d3a-4891-8d17-7fad155b5f72@www.fastmail.com>
In-Reply-To: <88132afe-0d3a-4891-8d17-7fad155b5f72@www.fastmail.com>
Date: Tue, 15 Jun 2021 00:36:35 +0100
Message-ID: <CAKxcST8tG88CWpukyUeHvRCkxReS2STDSdGGPiR96ovfddCL9A@mail.gmail.com>
To: Larry Garfield <larry@garfieldtech.com>
Cc: php internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000acdc7b05c4c254cd"
Subject: Re: [PHP-DEV] Allow objects in define()
From: dragoonis@gmail.com (Paul Dragoonis)

--000000000000acdc7b05c4c254cd
Content-Type: text/plain; charset="UTF-8"

On Mon, Jun 14, 2021 at 7:47 PM Larry Garfield <larry@garfieldtech.com>
wrote:

> On Mon, Jun 14, 2021, at 9:35 AM, Nikita Popov wrote:
> > Hi internals,
> >
> > With the introduction of enums, it's now possible for constants to hold
> > objects. However, this works only when using the `const X = Y` syntax,
> not
> > when using `define('X', Y)`, which still excludes objects.
> >
> > I've submitted https://github.com/php/php-src/pull/7149 to relax the
> > restriction. This means that define() would accept everything apart from
> > recursive arrays.
> >
> > An alternative here would be to allow define() to only accept enum
> objects
> > in particular, but not other objects. I would prefer not to do that, as
> > such a restriction would be rather arbitrary, now that the technical work
> > to support objects has been done. (PS: Please keep the difference between
> > mutability and interior mutability in mind.)
> >
> > Regards,
> > Nikita
>
> Sounds fine to me.  define() is runtime anyway so it should support
> runtime values.
>

Sounds sensible to me too. Good enhancement.


>
> --Larry Garfield
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: https://www.php.net/unsub.php
>
>

--000000000000acdc7b05c4c254cd--