Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124471
Precedence: bulk
MIME-Version: 1.0 (Mimestream 1.3.6)
References: <CAOWwgpmbq5VRrZQvUXDsKiNK4r6+bFA4VxnjQ_U=h8T9r0o3DA@mail.gmail.com>
 <ab83af79-0669-47dd-a3cb-ab72327ae174@bastelstu.be> <CAP1Jc13JWVg99AULgzrGXcoCWJtJZGz+nr_VoyUocw8n5n5sfQ@mail.gmail.com>
 <1118bbcd-a7b4-47bf-bf35-1a36ab4628e1@bastelstu.be> <CAP1Jc11vNiazMRL8Y-4NcJhsEkg6kgFNKLymTGfEPYRv114oTA@mail.gmail.com>
 <45847b93-02bf-459f-bcd2-81ba35a12c24@bastelstu.be> <CAOWwgpnt77-Sybzc3hcnFEui9t8wAv03NAov4xCK9ib_VRZ0JA@mail.gmail.com>
 <46bd4098-2936-4e46-98e9-fe55118325c2@bastelstu.be> <CAOWwgpkCy++W1C2t0PnkZ71S8vq1B734ZNPGw2-OHjmrwO3L4A@mail.gmail.com>
 <61ab36bc-b045-452a-84e0-87367d4c680e@bastelstu.be> <CAEPJduk6m1gdPOrsXYbsc+RLuHeEBav3pGwQO+PHs4UGGqtzhg@mail.gmail.com>
 <07e065f2-8f64-4bad-9a98-51f4eaf63ddb@app.fastmail.com> <CAOWwgpngQtg2NRW6M6RwgSH+OS1iipewHBmM5c36P2MxQGX78A@mail.gmail.com>
 <2a0a4650-c2c5-4c6d-ad3a-25365b3391b2@bastelstu.be> <CAOWwgpm9FLy_SNt7HLtJh5sxtfwjZMMN6pOKioDqsJc+GgD-sA@mail.gmail.com>
 <CAOWwgpnwoPH_rkJreh8v+X8WagzcCJ_puT1RWC0c717GMj5Onw@mail.gmail.com>
 <CAOWwgp=uwDebrx-=H8yP1Dgwf9ji=v_qagw9aAxbDJcPvCqKNw@mail.gmail.com>
 <b096b1d3-4414-4b53-a3b0-3a87c8dd7ff3@app.fastmail.com> <CAOWwgp=88jWSR-roWTrkkgwVgQvLQYbkTGSGAr6ASPs6s7s_4g@mail.gmail.com>
In-Reply-To: <CAOWwgp=88jWSR-roWTrkkgwVgQvLQYbkTGSGAr6ASPs6s7s_4g@mail.gmail.com>
Date: Thu, 18 Jul 2024 03:18:30 -0700
Message-ID: <CAGKNXQFEwJts2CERgMVmK9JTXubqcy1cmUd-=xD98X_dAciNsA@mail.gmail.com>
Subject: Re: [PHP-DEV] [RFC] Lazy Objects
To: Nicolas Grekas <nicolas.grekas+php@gmail.com>
Cc: php internals <internals@lists.php.net>, Larry Garfield <larry@garfieldtech.com>
Content-Type: multipart/alternative; boundary="000000000000869a07061d82e641"
From: phofstetter@sensational.ch (Philip Hofstetter)

--000000000000869a07061d82e641
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

 Hi,

Minor point: Why is the $initializer return type null, instead of void?  I
>> don't see a purpose to allowing an explicit null return and nothing else=
.
>>
>
> Updated to use "void". Both would work :)
>
>
Super minor nitpick: You have updated the prototype, but not the
explanation text which still says:

When initialization is required, the $initializer is called with the object
> as first parameter. The initializer should initialize the object, and mus=
t
> return null (or void). See the =E2=80=9CInitialization Sequence=E2=80=9D =
section.
>

However, given the `:void` return type, you can=E2=80=99t `return null` - t=
hat
would be a fatal error.

The phrase should probably be

When initialization is required, the $initializer is called with the object
> as first parameter. The initializer should initialize the object. See the
> =E2=80=9CInitialization Sequence=E2=80=9D section.
>

Philip

--000000000000869a07061d82e641
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<html><body><div dir=3D"ltr">
    Hi,</div><div dir=3D"ltr"><br clear=3D"all"></div><div class=3D"gmail_q=
uote"><blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;b=
order-left:1px solid rgb(204,204,204);padding-left:1ex" type=3D"cite"><div =
dir=3D"ltr"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" st=
yle=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padd=
ing-left:1ex">Minor point: Why is the $initializer return type null, instea=
d of void?=C2=A0 I don&#39;t see a purpose to allowing an explicit null ret=
urn and nothing else.<br></blockquote><div><br></div><div>Updated to use &q=
uot;void&quot;. Both would work :)<br></div><div><br></div></div></div>

    </blockquote><br>
</div><div class=3D"gmail_quote" dir=3D"ltr">Super minor nitpick: You have =
updated the prototype, but not the explanation text which still says:</div>=
<div class=3D"gmail_quote" dir=3D"ltr"><br></div><blockquote style=3D"margi=
n:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-le=
ft-color:rgb(204,204,204);padding-left:1ex" class=3D"gmail_quote" type=3D"c=
ite"><div class=3D"gmail_quote" dir=3D"ltr">When initialization is required=
, the $initializer is called with the object as first parameter. The initia=
lizer should initialize the object, and must return null (or void). See the=
 =E2=80=9CInitialization Sequence=E2=80=9D section.</div></blockquote><br>
   =20
<div dir=3D"ltr">However, given the `:void` return type, you can=E2=80=99t =
`return null` - that would be a fatal error.</div><div dir=3D"ltr"><br></di=
v><div dir=3D"ltr">The phrase should probably be</div><div dir=3D"ltr"><br>=
</div><blockquote style=3D"margin:0px 0px 0px 0.8ex;border-left-width:1px;b=
order-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"=
 class=3D"gmail_quote" type=3D"cite"><div dir=3D"ltr">When initialization i=
s required, the $initializer is called with the object as first parameter. =
The initializer should initialize the object. See the =E2=80=9CInitializati=
on Sequence=E2=80=9D section.</div></blockquote><br><div dir=3D"ltr">Philip=
</div></body></html>

--000000000000869a07061d82e641--