On Fri, 6 Jun 2003, Eugene van der Pijll wrote:

> The changes are:
> - I moved the _generate_spans_until_match() method (that is included in
>   perhaps half of all DT::TZ's, and is a largish piece of code) to
>   TimeZone. This saves about 100kB.
>
> - I changed the format of the dst changes, from
>
>       {
>         'short_name' => 'LMT',
>         'utc_end' => '57875470828',
>         'utc_start' => DateTime::TimeZone::NEG_INFINITY,
>         'local_end' => '57875472000',
>         'is_dst' => 0,
>         'offset' => 1172,
>         'local_start' => DateTime::TimeZone::NEG_INFINITY
>       },
>
>   to
>
>       
> ['LMT','57875470828',DateTime::TimeZone::NEG_INFINITY,'57875472000',0,1172,DateTime::TimeZone::NEG_INFINITY]
>
>   This may make the generated file more obfuscated, but it does make a
>   difference of 2Mb. (And I find the new format clearer in a way: it is
>   easier to get an overview of all the changes.)

This is a good start, but it didn't go far enough ;)

You still have it turning those array refs back to hash refs.  Might as
well just go for it and use arrays internally.  It's faster and uses less
memory.

Anyway, I think I've got that done.


-dave

/*=======================
House Absolute Consulting
www.houseabsolute.com
=======================*/

Reply via email to