Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:124922
Precedence: bulk
MIME-Version: 1.0
References: <CAPyj-LB11wiZyHV8=7fEs=J==tDO+m+z4qRsXRroPnqXzNZvvg@mail.gmail.com>
 <9e1068ae-593e-40bc-91d7-dd63545d9e60@gmx.de> <CAAKU0ukMEAdCYQmti=aqM7QyjtnuzOx6fCd0aWGGf6EN3CRZBw@mail.gmail.com>
 <CAPyj-LCArhAARxFrCh+9C4T2oj=3EL1vy98+whuStjsBHJAMCQ@mail.gmail.com> <CAAKU0uk+wwDp0Ujz7CRO-S8SA-qEaGrX7w-eUx5zFCTsUf8cOQ@mail.gmail.com>
In-Reply-To: <CAAKU0uk+wwDp0Ujz7CRO-S8SA-qEaGrX7w-eUx5zFCTsUf8cOQ@mail.gmail.com>
Date: Tue, 13 Aug 2024 17:56:51 +0200
Message-ID: <CAPyj-LDhKRuv-dfHbG-e32MChtQmXHcaH_UAFemsGH+4h9RBew@mail.gmail.com>
Subject: Re: [PHP-DEV] Require C11 in PHP 8.4
To: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
From: tovilo.ilija@gmail.com (Ilija Tovilo)

Hi Levi

On Tue, Aug 13, 2024 at 5:00=E2=80=AFPM Levi Morrison
<levi.morrison@datadoghq.com> wrote:
>
> On Tue, Aug 13, 2024 at 8:17=E2=80=AFAM Ilija Tovilo <tovilo.ilija@gmail.=
com> wrote:
> >
> > Just to state it officially: You object to switching to C11 in 8.4? In
> > that case, we'll have to postpone.
>
> I don't object. I think it's smarter to wait, and do more than just
> redeclare a few typedefs. But I won't block it.

Thank you for the clarification. In that case, I'd say let's go
forward with the proposal. I agree that we should be making use of
more C11 features in the future, once the opportunity presents itself.

> If only documentation changes, then I think the risk is lower. We
> definitely should not be doing configure checks in 8.4 with how little
> time there is, except maybe _one_ that attempts to redeclare a typedef
> to see if it compiles, and give a nicer error message.

I agree. I will create a PR to provide a ./configure time error
message if the compiler doesn't support typedef redeclarations.
Adopting C11 does mean we can start using other C11 features too,
though that should naturally happen only on (future) master rather
than 8.4, given that ABI freeze is only a little over a month away.

Ilija