On Mon, Oct 4, 2010 at 10:42 AM, Derick Rethans <der...@php.net> wrote:
> On Mon, 4 Oct 2010, Pierre Joye wrote:
>
>> On Mon, Oct 4, 2010 at 1:27 AM, Stas Malyshev <smalys...@sugarcrm.com> wrote:
>> >
>> >> It looks like a sub optimal choice to have used string constants
>> >> instead of integer. However it could be still possible to define new
>> >> constants as numeric. It is then possible to do whatever needs to be
>> >> done as post or pre ops for the respective constants.
>> >
>> > I'm not sure what integers have to do with it? The constants define date
>> > formats that are in common use, RFC2616 is one of the commonest on the web
>> > and we don't have a constant for it...
>>
>> I mean in ext/date and as a reply to Derick, not your request which is
>> totally valid. A date time object has the timezone information. If the
>> constants were integers, it would be very straightforward to do some
>> operations before calling the formatting functions depending on a
>> given predefined format. It should still be possible to do it by
>> testing the string contents (strncmp), but that's not very clean.
>
> Stop talking about something you don't know anything about, please.
> Those constants are not *one* format letter, they are many. Maybe you
> could have tried this:
>
> echo DateTime::RFC822, "\n";

Maybe you could read what I wrote instead of replying in such stupid
manner, that could lead the discussions in a constructive direction,
thanks.

Key parts were:

1. using integer makes such features easier to implement (basic good
practice/programming 101)
2. use strncmp to compare the format (yes, strncmp supports many characters)

2. is bad as it has the side effects of applying to a format given
manually even if one did not intend to use it this way, it could have
BC issues but I'm not sure at all if such format exists.

Cheers,
-- 
Pierre

@pierrejoye | http://blog.thepimp.net | http://www.libgd.org

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to