On Mon, 15 Feb 2016, Bert Freudenberg wrote:
On 15.02.2016, at 10:17, marcel.taeumel Marcel.Taeumel@hpi.de wrote:
Hi Bert,
this was just a regression. There has always been this check in the past for Morphic projects and still today for MVC projects.
Ah, so we lost the check at some point?
If you would have used VM or OS startup time, this would still be problematic after an overflow. (Hence the comment about snapshotting). So, this fix does not directly address the discussion about synching #millisecondClockValue to wall clock.
I still think it should answer milliseconds since startup. Why would we change that?
Eliot changed it recently. Probably to avoid the rollover issues. The correct fix would be to use to UTC clock instead of the local one in Time class >> #millisecondClockValue.
Currently this change also affects performance (down to 8-10% of the previous implementation), because of the creation of multiple LargeIntegers.
Levente
- Bert -