TimeZone2 - first impressions

Mind Gravy » Blog Archive » links for 2008-05-17

Mind Gravy » Blog Archive » links for 2008-05-17 — Sat, 17 May 2008 12:33:33 GMT

Pingback from Mind Gravy » Blog Archive » links for 2008-05-17

re: TimeZone2 - first impressions

bill — Thu, 05 Oct 2006 01:21:14 GMT

more thoughts on 8.

As it stands the caller has to know when to call TimeZone2.ClearCachedData, which is pretty hard for them to do while only using the framework.

Looking at what is cleared in that, the only one of any real significnace I think is the Local property. that is, through standard window Ui's it's easy ot change the current time zone. UTC won't change, and althohg any cached system time zonez might change, when they are changed, it's not a standard windows operation.

So the question is is the overhead of calling GetDynamicTimeZoneInformation and comparing the data with the cached Local timezone2 so significant that we cut functionality for the sake of performance ?

Personally I would think that functionality is more important than performance here, and should the perf be an issue, the caller can cache TimeZone2.Local themselves. hence haivng Timezone2.Local *always* check to see if ti has changed would be the better design IMNSHO :)

re: TimeZone2 - first impressions

bill — Thu, 05 Oct 2006 00:54:07 GMT

Hi Kathy,

Re 5, oh yes it is !!

Dim tz1 As TimeZone2 = TimeZone2.FindSystemTimeZoneById("AUS Eastern Standard Time")

Dim tz2 As TimeZone2 = TimeZone2.FindSystemTimeZoneById("AUS Eastern Standard Time")

If tz1 Is tz2 Then ' returns False

The problem is inside TryGetTimeZone, jsut look for all the "new"'s in there ;)

RE 6: it's because of 5 :)

Re 8: Cool !! Sorry I didn't notice it, my bad :(

Thanks,

Bill.

re: TimeZone2 - first impressions

KathyKam — Wed, 04 Oct 2006 20:15:24 GMT

Hey @Head,

This is some great feedback! I'll probably address all of them in my next posting! :)

There is some quick responses to your points above:

5: FindSystemTimeZoneById is not reaturing a fresh instance everytime.

6: The behaviour you are seeing is a known issue that will be fixed in the RTM version.

8: TimeZone2.ClearCachedData allows you to clear the cached Data.

Thanks,

Kathy