368723 - Revisit architecture to upgrade timezone ids on the fly

Status: RESOLVED DUPLICATE of bug 363191

(Calendar :: Internal Components, defect)

Description

[Matthew (lilmatt) Willis]

In bug 368121, mvl wrote the following:

>>Index: calendar/base/src/calDateTime.h
>> class calDateTime : public calIDateTime,
>>                     public nsIXPCScriptable
>>-    nsCString mTimezone;
>>+    calTzId mTimezone;
>Why-oh-why is this a new class. What was wrong with the normal string?

We subclassed the string so that we could easily hook into mTimezone.Assign() and do a check for tzid "freshness" there.

This bug is to allow for revisiting that decision for 0.5 and future releases.

Comment 1

[Matthew (lilmatt) Willis]

Not going to make the 0.5 train.

Comment 2

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Clint, has there been any change in the way timezones work in the latest 0.8pre nightlies, or if the future outlook has changed?

Comment 3

[Daniel Boelzle [:dbo]]

The mentioned code doesn't exist anymore in this form. We will keep care of updating timezones (and ids) with bug 363191.