<div dir="ltr">Why not ChronologyTests?<br></div><div class="gmail_extra"><br><div class="gmail_quote">2016-02-18 2:33 GMT+01:00 David T. Lewis <span dir="ltr">&lt;<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>&gt;</span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I would like to move Chronology from &#39;Kernel-Chronology&#39; to a new package<br>
called &#39;Chronology&#39;, and move &#39;KernelTests-Chronology&#39; to a new package<br>
called &#39;Tests-Chronology&#39;.<br>
<br>
Rationale:<br>
<br>
- Chronology is a large package that is functionally separable from the<br>
rest of the things in Kernel.<br>
<br>
- I have a UTC based variation of Chronology that addresses a number of<br>
issues that I think may be of long term benefit, and that prevent the kinds<br>
of issues that we are currently seeing in trunk. I would like to offer this<br>
for review in the inbox, but that is not practical with the current<br>
packaging.<br>
<br>
Would anyone object to this change to package structure?<br>
<br>
Thanks,<br>
Dave<br>
<br>
<br>
On Wed, Feb 17, 2016 at 05:13:05PM -0800, tim Rowledge wrote:<br>
&gt; We can see the primitives involved with this do it -<br>
&gt;  {SystemNavigation default browseMessageList: #(<br>
&gt; &#39;Time class primMicrosecondClock 117&#39;<br>
&gt; &#39;Time class primMillisecondClock 135???<br>
&gt; &#39;Time class eventMillisecondClock 135???<br>
&gt; &#39;Time class primPosixMicrosecondClockWithOffset 117???<br>
&gt; &#39;Time class primUTCMicrosecondClock 240???<br>
&gt; &#39;Time class localMicrosecondClock 241???<br>
&gt; &#39;Time class utcMicrosecondClock 240???<br>
&gt; &#39;Time class primSecondsClock 137???<br>
&gt; &#39;Time class primMillisecondClockMask 117???<br>
&gt; &#39;Time class primLocalMicrosecondClock 241&#39;) name: &#39;time prims???}<br>
&gt;<br>
&gt; a) primMicrosecondClock is not sent in a 5.0 image and seems to only be implemented in a plain interpreter VM.<br>
&gt; b) primMillisecondClock/135 is not sent in a 5.0 image<br>
&gt; c) eventMillisecondClock/135 is used for the faked-up events required for double-click handling<br>
&gt; d) primPosixMicrosecondClockWithOffset is not sent in 5.0<br>
&gt; e) primUTCMicrosecondClock/240 is not sent in 5.0 - would return uSec value in raw<br>
&gt; f) localMicrosecondClock/241 is used in 5.0 in particular from Time now (as per the ClockMorph)<br>
&gt; g) utcMicrosecondClock/240 is used in 5.0 for Delays and timing type cases<br>
&gt; h) primSecondsClock/137 isn???t used in 5.0<br>
&gt; i) primMillisecondClockMask isn???t used in 5.0<br>
&gt; j) primLocalMicrosecondClock/241 isn???t used in 5.0<br>
&gt;<br>
&gt; So we have quite a collection of mixedup terminology and unused methods to clear up!<br>
&gt;<br>
&gt; Looking at it from the prims point of view -<br>
&gt;<br>
&gt; Prim 135 is needed in order to provide the same timebase as the events coming in, so we can???t drop that, and it affects nothing else.<br>
&gt;<br>
&gt; Prim 240 is for no-TZ microseconds and is used in Delay etc but crucially as part of DateAndTime&gt;now which is why we are having a problem in the latest updated 5.0 with no TZ/Locale being used.<br>
&gt; Prim 241 is used by Time now etc and includes the TZ offset, which is why Time now returns correct wall-clock time.<br>
&gt;<br>
&gt; The others are no longer used and could be removed except for the primPosixMicrosecondClockWithOffset which we ought to make use of so that the TZ is reported back.<br>
&gt;<br>
&gt; tim<br>
<span class="HOEnZb"><font color="#888888">&gt; --<br>
&gt; tim Rowledge; <a href="mailto:tim@rowledge.org">tim@rowledge.org</a>; <a href="http://www.rowledge.org/tim" rel="noreferrer" target="_blank">http://www.rowledge.org/tim</a><br>
&gt; Useful Latin Phrases:- Quantum materiae materietur marmota monax si marmota monax materiam possit materiari? = How much wood would a woodchuck chuck if a woodchuck could chuck wood?<br>
&gt;<br>
&gt;<br>
<br>
</font></span></blockquote></div><br></div>