Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124852
Precedence: bulk
MIME-Version: 1.0
References: <CAPyj-LB11wiZyHV8=7fEs=J==tDO+m+z4qRsXRroPnqXzNZvvg@mail.gmail.com>
 <9e1068ae-593e-40bc-91d7-dd63545d9e60@gmx.de> <CAPyj-LCpC4KzUSWK08ehEVmzarZWacGnJ0uDJrF9aAtCqfzVQw@mail.gmail.com>
In-Reply-To: <CAPyj-LCpC4KzUSWK08ehEVmzarZWacGnJ0uDJrF9aAtCqfzVQw@mail.gmail.com>
Date: Sun, 11 Aug 2024 17:27:38 +0200
Message-ID: <CAML1Pf5EndWtNkE8=ut6MLSushaosq95+k2FBym0cHKeX3wYfw@mail.gmail.com>
Subject: Re: [PHP-DEV] Require C11 in PHP 8.4
To: Ilija Tovilo <tovilo.ilija@gmail.com>
Cc: PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary="000000000000c0cd6c061f6a0466"
From: giovanni@giacobbi.net (Giovanni Giacobbi)

--000000000000c0cd6c061f6a0466
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Sun, 11 Aug 2024 at 16:12, Ilija Tovilo <tovilo.ilija@gmail.com> wrote:

> Hi Christoph
>
> On Sat, Aug 10, 2024 at 2:19=E2=80=AFPM Christoph M. Becker <cmbecker69@g=
mx.de>
> wrote:
> >
> > On 01.08.2024 at 23:57, Ilija Tovilo wrote:
> >
> > So skimming the whole discussion[1] it seems that most are generally
> > fine with bumping the requirements to C11, except for Giovanni Giacobbi
> > (whose draft PR[2] had no further discussion so far), and maybe for som=
e
> > uncertainties regarding some less used compilers.
>
> Giovanni's remark that this would impact many people was challenged by
> Jakub [1] which didn't get a response. I believe it's safe to assume
> that this isn't the case.
>
>
I'd like to remark that this is not a decision about renouncing some
valuable feature in order to support older compilers, but rather about
applying a patch shy of ~100 lines where half of the changes should be
considered bugfixes.

If you think it would be acceptable to have an additional header file
pre-declaring the typedefs, it would solve the only standing issue that
prevents the entire codebase to be C99 compliant. Isn't it worth it? The
more systems that can bild PHP, the better :-)

Also this is not a binding decision: If in the future there is a real need
to increase the requirement to C11, you can always do that.

--000000000000c0cd6c061f6a0466
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr">On Sun, 11 Aug 2024 at 16:12, Ilija Tovil=
o &lt;<a href=3D"mailto:tovilo.ilija@gmail.com">tovilo.ilija@gmail.com</a>&=
gt; wrote:<br></div><div class=3D"gmail_quote"><blockquote class=3D"gmail_q=
uote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,2=
04);padding-left:1ex">Hi Christoph<br>
<br>
On Sat, Aug 10, 2024 at 2:19=E2=80=AFPM Christoph M. Becker &lt;<a href=3D"=
mailto:cmbecker69@gmx.de" target=3D"_blank">cmbecker69@gmx.de</a>&gt; wrote=
:<br>
&gt;<br>
&gt; On 01.08.2024 at 23:57, Ilija Tovilo wrote:<br>
&gt;<br>
&gt; So skimming the whole discussion[1] it seems that most are generally<b=
r>
&gt; fine with bumping the requirements to C11, except for Giovanni Giacobb=
i<br>
&gt; (whose draft PR[2] had no further discussion so far), and maybe for so=
me<br>
&gt; uncertainties regarding some less used compilers.<br>
<br>
Giovanni&#39;s remark that this would impact many people was challenged by<=
br>
Jakub [1] which didn&#39;t get a response. I believe it&#39;s safe to assum=
e<br>
that this isn&#39;t the case.<br>
<br></blockquote><div><br></div><div>I&#39;d like to remark that this is no=
t a decision about renouncing some valuable feature in order to support old=
er compilers, but rather about applying a patch shy of=C2=A0~100 lines wher=
e half of the changes should be considered bugfixes.</div><div><br></div><d=
iv>If you think it would be acceptable to have an additional header file pr=
e-declaring the typedefs, it would solve the only standing issue that preve=
nts the entire codebase to be C99 compliant. Isn&#39;t it worth it? The mor=
e systems that can bild PHP, the better :-)<br></div><div><br></div><div>Al=
so this is not a binding decision: If in the future there is a real need to=
 increase the requirement to C11, you can always do that.</div><div><br></d=
iv></div></div>

--000000000000c0cd6c061f6a0466--