Martin Eisenhardt
Research assistant, soon-to-be Senior Dev · Not a member
Martin Eisenhardt's latest activity
Sunday, May 18 2008
Friday, May 16 2008
-
Martin Eisenhardt commented at 3:19 PMOK, I got the point: This functionality is better implemented in an i18n plugin. No worries.
But: You should really think about what kind of impression it mak...
Thursday, May 15 2008
-
Martin Eisenhardt commented at 4:25 PMHi Geoff,
ad 1.) I agree that a Pacific time zone group is probably nothing that will be used very much. I do see much uses for european_zones or asian_zones....
-
Martin Eisenhardt commented at 10:24 AMHello Star Trader,
thank you for the review and valuable input. I do agree that further discussion as to which time zones shall be grouped together is necessa...
-
Martin Eisenhardt commented at 3:34 PMHi Joshua,
thank you for the advice. It is the same I had in mind. Together with this comment, I upload an updated patch the pre-computes a US_ZONES constant,...
-
Martin Eisenhardt commented at 2:55 PMWhile regular expressions are by far the fastest way to search for patterns, they nevertheless are slow compared to using pre-computed values.
I really do lik...
-
Martin Eisenhardt commented at 12:02 PMHi Pratik, thank you for your review and comment.
Ad 1.) OK, I can understand that this does not meet your personal coding standard, or the coding standard of...
Wednesday, May 14 2008
-
Martin Eisenhardt created the ticket at 7:45 PMThe us_zones convenience method in TimeZone generates the list of TimeZone objects in the U.S. every time it is called. This is inefficient, especially in appl...
-
Martin Eisenhardt created the ticket at 6:38 PMIn the TimeZone class in ActiveSupport, there is a convenience method us_zones that dynamically creates a list of all time zones for the U.S.
As is easy to se...