Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:114658
MIME-Version: 1.0
References: <1815820036.3212683.1621007783673.JavaMail.zimbra@dafert.at>
In-Reply-To: <1815820036.3212683.1621007783673.JavaMail.zimbra@dafert.at>
Date: Fri, 28 May 2021 16:51:17 +0200
Message-ID: <CAF+90c923S_sPfCCXoih1yg8OaePxALCj6urR3TszYOuqKhTgw@mail.gmail.com>
To: Mel Dafert <mel@dafert.at>
Cc: internals <internals@lists.php.net>, Sara Golemon <pollita@php.net>
Content-Type: multipart/alternative; boundary="000000000000096bcd05c36503b2"
Subject: Re: [PHP-DEV] [VOTE] Add IntlDatePatternGenerator
From: nikita.ppv@gmail.com (Nikita Popov)

--000000000000096bcd05c36503b2
Content-Type: text/plain; charset="UTF-8"

On Fri, May 14, 2021 at 5:56 PM Mel Dafert <mel@dafert.at> wrote:

> Hi Internals,
> I have opened the vote on
> https://wiki.php.net/rfc/intldatetimepatterngenerator.
> I will close it on 2021-05-28.
>
> For previous discussion see https://externals.io/message/113831 and
> https://externals.io/message/114124.
>
> Regards,
> Mel
>

A bit late, but I have a small note on this RFC:

It's ... checks calendar ... the year 2021. Do we *really* need to add a
procedural mirror APIs, especially with such auspicious function names as
datepatterngenerator_get_best_pattern?

I believe the procedural APIs are considered legacy APIs, and we are
intentionally not adding them for new functionality. For example, the most
recent intl addition (IntlChar) does not come with procedural APIs.

Regards,
Nikita

--000000000000096bcd05c36503b2--