Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:101932
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: error (pb1.pair.com: domain dqxtech.net from 209.85.215.49 cause and error)
MIME-Version: 1.0
In-Reply-To: <CAA69SC48P1zn+-0G6tQyR6TGrrDBhVT+mg6zKQYpNZkU2+2HmA@mail.gmail.com>
References: <CAA69SC79fvaKAT=x=zoUfaqpiK78q68_cdR0vonJAm9FKAwJeQ@mail.gmail.com>
 <CAF+90c-=XtkjmrNV-am3RZ4+XhgJ0WgFCV5Yevjebz2UUh=y3Q@mail.gmail.com>
 <CALKiJKoqPpByQNVMG-3V2Jqjnf=6BkfqEKj96grSdqiJgw1qGg@mail.gmail.com>
 <CAA69SC5z5yxhAHFdbpf_RYBcKia56MZBXuAEk55BhPuiCnFRYw@mail.gmail.com>
 <CAA69SC5irBCAK5k7Adj7-jd-zbTuk+--WhVi0u7e8CnZt81oFw@mail.gmail.com>
 <CA+kxMuTa6zucL2O+n3hf251XABO=UfpFtVNvUA_FXFx6pWz=bw@mail.gmail.com>
 <CAA69SC6yDSJ4jD7By+iHVuBe8x2w_dH+wfiC1EmwkSw=LTNxFQ@mail.gmail.com>
 <CAA69SC6JQ9viE+Z-nxPtOfZVrScT7Cu3UBFFZ1R3WLREq-5YjQ@mail.gmail.com>
 <CAA69SC72se=ZyLhadAUSOhUSaq8pQ=rZT4Ufcx=PKsmS7vqrDw@mail.gmail.com>
 <CAA69SC7ewuka5Oq5yyjSL7X3Jq249WKbkhyq2XXXwBWBOUt6ZA@mail.gmail.com>
 <C7D5FF76-63FA-4A27-8E98-1D030D7D87F0@gmail.com> <CAA69SC7B2VFkj=J4zY+tWpCdSaBxX+TKFu=zE2oJNauNxgz_xA@mail.gmail.com>
 <D9002600-2954-4C47-85D0-E617DCDF1655@gmail.com> <CAGU04mKd_R5eYyogT1-MRzr6xMtAFEKDmpPdq0uxs3vpB9pmOA@mail.gmail.com>
 <CAA69SC4vkpbTNQSeTfWxWPuqmzCspgb2Yp2S2WcVY0433jM-8A@mail.gmail.com>
 <CAA69SC4qntSeNebXupbum-LEqHyLgwG+TDmsWtPVK6xTvK1h7w@mail.gmail.com>
 <CAA69SC7qBkEA9tQnDqbeWsE4XTt21qOKhDmZNv3293LiUj80SA@mail.gmail.com>
 <CAA69SC7ec7z3U5QvTH_GMy9w9p21SCZYykgDN4EvbN-xwg654g@mail.gmail.com> <CAA69SC48P1zn+-0G6tQyR6TGrrDBhVT+mg6zKQYpNZkU2+2HmA@mail.gmail.com>
Date: Mon, 26 Feb 2018 16:11:39 +0100
Message-ID: <CAH0Uv3HzZvyVorS6LN+u6sNUYPjr1viqo8fN7ZB7NVakfT7Q1w@mail.gmail.com>
To: =?UTF-8?Q?Silvio_Mariji=C4=87?= <marijic.silvio@gmail.com>
Cc: PHP Internals List <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Subject: Re: [PHP-DEV][RFC][DISCUSSION] Immutability
From: andreas@dqxtech.net (Andreas Hennings)

Hello,
my thoughts.
For me personally the proposed feature would not be very useful.
i like immutable objects. But for me the pattern of choice is private
properties + constructor + optional "wither" methods.

The "wither" methods create a clone of the object, then set a property
value on the clone, then return the clone.
This would already not work if the property is marked as immutable
with the proposed feature.

Also static method constructors that write directly on the object
would not work, because technically they are not part of the
constructor.

For private properties the proposed feature would add little benefit.
Any private property can be made practically immutable by writing the
class accordingly: Not having setters, etc.

Where this feature would become useful is if someone prefers public
properties over getter methods.
But again, this would prevent any unconventional constructor, like
withers or static factories.

I think a better and more useful feature would be read-public, write-privat=
e.
Maybe like this? https://wiki.php.net/rfc/readonly_properties
(For some reason I cannot open the RFC wiki pages. Maybe they are down?)

This would achieve the same goal: Making a property practically immutable.
But it would still allow unconventional constructors.
It would be the implementor's job to decide which method should be
allowed to modify a property, and which shouldn't.



On 26 February 2018 at 13:52, Silvio Mariji=C4=87 <marijic.silvio@gmail.com=
> wrote:
> Currently the build is failing in some parts of the codebase that are
> obviously affected in some way.
> I ran valgrind for couple of failing tests and I got numerous reports abo=
ut
> memory leaks and conditional jumps over uninitialized values.
> Not sure what is the reason for that, I had merge conflict with upstream
> master couple days ago, might be a reason.
>
> I would appreciate if anyone could give me hand on this ?
>
> On Feb 26, 2018 1:48 PM, "Silvio Mariji=C4=87" <marijic.silvio@gmail.com>=
 wrote:
>
>> Currently the build is failing in some parts of the codebase that are
>> obviously affected in some way.
>> I ran valgrind for couple of failing tests and I got numerous reports
>> about memory leaks and conditional jumps over uninitialized values.
>> Not sure what is the reason for that, I had merge conflict with upstream
>> master couple days ago, might be a reason.
>>
>> I would appreciate if anyone could give me hand on this ?
>>
>> On Feb 26, 2018 1:44 PM, marijic.silvio@gmail.com wrote:
>>
>> Yes, I've also took that into consideration when choosing keyword.
>>
>> On Feb 26, 2018 11:35 AM, "Crocodile" <crocodile2u@gmail.com> wrote:
>>
>> Is "value" or "immutable" going to become a new reserved word? Ain't we
>> going to have some BC breaks because of that? If so, then "value" is goi=
ng
>> to bring more BC breaks then "immutable".
>>
>> On Sun, Feb 25, 2018 at 8:02 PM Paul Jones <pmjones88@gmail.com> wrote:
>>
>>>
>>>
>>> > On Feb 25, 2018, at 12:59, Silvio Mariji=C4=87 <marijic.silvio@gmail.=
com>
>>> wrote:
>>> >
>>> > Here is link to tweet https://twitter.com/SilvioMari
>>> jic/status/965564630071300096
>>>
>>> After having read that, I think "immutable" is still perfectly reasonab=
le.
>>>
>>>
>>> --
>>> Paul M. Jones
>>> pmjones@pmjones.io
>>> http://paul-m-jones.com
>>>
>>> Modernizing Legacy Applications in PHP
>>> https://leanpub.com/mlaphp
>>>
>>> Solving the N+1 Problem in PHP
>>> https://leanpub.com/sn1php
>>>
>>>
>>>
>>>
>>> --
>>> PHP Internals - PHP Runtime Development Mailing List
>>> To unsubscribe, visit: http://www.php.net/unsub.php
>>>
>>> --
>> Best regards,
>> Victor Bolshov
>>
>>
>>
>>