Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:59317
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.213.170 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <op.wb4p5lxsidpuyk@slws007.slhq.int>
References: <op.wb4p5lxsidpuyk@slws007.slhq.int>
Date: Mon, 2 Apr 2012 11:13:02 +0200
Message-ID: <CAEZPtU63cgkFLEMOjAAd8kDXY8adFymPcW2f2Evi=A9r-GPzFw@mail.gmail.com>
To: Gustavo Lopes <glopes@nebm.ist.utl.pt>
Cc: "internals@lists.php.net" <internals@lists.php.net>, Stas Malyshev <smalyshev@sugarcrm.com>
Content-Type: text/plain; charset=ISO-8859-1
Subject: Re: [PHP-DEV] Addition of calendar to intl
From: pierre.php@gmail.com (Pierre Joye)

hi Gustavo,

Very nice job :)

The only thing I really dislike is the procedural API. The prefix
"intlcal_" is not very nice, but what I really doubt is the usefulness
of the procedural APIs for the Calendar resources. It makes little to
no sense to have it as it brings nothing in comparison to the OO
version.

I know we could have it to be consistent with other part of the intl
extension but ...

Cheers,

On Mon, Apr 2, 2012 at 10:35 AM, Gustavo Lopes <glopes@nebm.ist.utl.pt> wrote:
> Hi
>
> I have exposed ICU's Calendar API to PHP via the intl extension. It allows
> date calculations with Gregorian, Chinese, Coptic, Ethiopic, Hebrew, Indian,
> Islamic (civil/religious), Japanese, Persian, Taiwan and Thai Buddhist
> calendars. For a broader overview of its functionality, see:
>
> http://icu-project.org/apiref/icu4c/classCalendar.html
>
> I have committed already this change:
>
> https://github.com/php/php-src/commit/1c16c89b96bb4e02850b198b901feebdb8da2269
>
> Traditionally, these kinds of additions to the intl extension (that mostly
> exposed 1 to 1 the ICU API) have not been preceded (or even followed) by any
> discussion; however, Stas expressed the opinion this is not a proper course
> of action nowadays.
>
> Consequently, if anyone has any non-trivial substantive (as opposed to
> procedural) concern with this change, I will revert the merge and submit an
> RFC.
>
> --
> Gustavo Lopes
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php
>



-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org