VA 7.0.x - DateAndTime class>primitiveSystemOffset

[marten]

I just noticed, that the bug in this method is still not fixed - ok, its winter and it does not matter. But it should be fixed at last in 7.5.

I still think, that is would be much better, if we have a public open list of errors for this product ! Then we can also see, what error description has reached the developers and maintainers.

Marten

[solveig]

Hi, Marten,

You are correct. DateAndTime class>primitiveSystemOffset still does not return the proper offset during daylight savings time.

We still have the Windows workaround you sent us. I will post it here, but we need to address this issue on Unix platforms.

---start of DateAndTime class>primitiveSystemOffset method
primitiveSystemOffset
| daylightSavingInAction baseOffset dstOffset timezoneInformation |

"
<primitive: TzOffset>
^self primitiveFailed
"
timezoneInformation := OSTimeZoneInformation calloc: 1.
daylightSavingInAction := timezoneInformation getTimeZoneInformation.

baseOffset := timezoneInformation Bias.
dstOffset := timezoneInformation DaylightBias.

timezoneInformation free.

^(baseOffset
+ (daylightSavingInAction= PlatformConstants::TimeZoneIdDaylight
ifTrue:[ dstOffset]
ifFalse:[ 0 ])) * 60


---end of DateAndTime class>primitiveSystemOffset method

A public list of known errors makes for transparency but raises other issues. Would periodic beta releases with a list of fixes be useful to you?

Regards,
Solveig

[marten]

A public list of known errors makes for transparency but raises other issues. Would periodic beta releases with a list of fixes be useful to you?

Regards,
Solveig

Yes other issues may be there - but I can not imagine, that these issues are important for developers. Perhaps more for marketing persons or layers.

And no - a periodic beta will not help me much.


Marten